Project

General

Profile

Actions

Cleanup #47355

closed

mgr/dashboard: create directive for AuthStorage service

Added by Ernesto Puerta over 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Normal
Category:
Security & Auth
Target version:
% Done:

0%

Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

Right now, if a component wants to conditionally display its elements based on authorization/scopes criteria, it has to embed direct calls to the AuthService. That basically causes a tight coupling between components and the AuthStorage API (localStorage based).

By using a simple directive, we can move all references to AuthStorage to the HTML template, leave a cleaner component logic (no intermixed AuthStorage calls), and simplify unit tests (no need of 'mocking' auth/localstorage, just a directive).

Actions #1

Updated by Ernesto Puerta about 3 years ago

  • Translation missing: en.field_tag_list set to low-hanging-fruit
Actions #2

Updated by Ernesto Puerta about 3 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 145 to Security & Auth
Actions #3

Updated by Ngwa Sedrick Meh about 3 years ago

Hi, How can we resolve this?

Actions #4

Updated by Ngwa Sedrick Meh about 3 years ago

  • Pull request ID set to 41034
Actions #5

Updated by Alfonso Martínez almost 3 years ago

  • Status changed from New to Fix Under Review
  • Assignee set to Ngwa Sedrick Meh
  • Target version changed from v16.0.0 to v17.0.0
Actions #6

Updated by Ernesto Puerta almost 3 years ago

  • Status changed from Fix Under Review to Resolved
Actions

Also available in: Atom PDF