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

Add selector for auth role #165

Open
arynh opened this issue Feb 26, 2020 · 1 comment
Open

Add selector for auth role #165

arynh opened this issue Feb 26, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request frontend

Comments

@arynh
Copy link
Contributor

arynh commented Feb 26, 2020

Is your feature request related to a problem? Please describe.
When directing the role from the state to the props of a components, we use selectors for pretty much all information except the auth role, which we reference directly.

Describe the solution you'd like
Sticking to the selector pattern for working with Redux state might make future things easier in the case that we start seeing more information derived from the state.

@arynh arynh added enhancement New feature or request frontend labels Feb 26, 2020
@arynh arynh self-assigned this Feb 26, 2020
@arynh
Copy link
Contributor Author

arynh commented Feb 26, 2020

I'd like some input on this though because it might be entirely unnecessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request frontend
Projects
None yet
Development

No branches or pull requests

1 participant