Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Task/177493 create user store #1568

Merged
merged 5 commits into from
Nov 28, 2024
Merged

Conversation

diogo-costa-bjss
Copy link
Collaborator

@diogo-costa-bjss diogo-costa-bjss commented Nov 27, 2024

Description:

  • Migrate the last authentication store to the new format.
  • Remove the leftovers of old stores.
  • Removed on multiple places the use of normal variables into using directly the store signal. Some of the most changed:
    • isAdminType now is ctx.user.isAdmin()
    • isInnovatorType now is ctx.user.isInnovator()
    • isAssessmentType now is ctx.user.isAssessment()
    • isAccessorRole now is ctx.user.isAccessor()
    • isQualifyingAccessorRole now is ctx.isQualifyingAccessor
    • isAccessorType now is ctx.user.isAccessorType -> If is a QA || A
    • Added new ctx.user.isAccessorOrAssessment() that is a shorthand for this.isAccessor() || this.isAssessment();

Related tickets:

- Created the store and migrated all authentication store methods
- Changed all places were authentication store was being used
@diogo-costa-bjss diogo-costa-bjss merged commit b3ea3a3 into develop Nov 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants