-
Notifications
You must be signed in to change notification settings - Fork 23
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
Modernize redux: aclDetailsSlice #251
Modernize redux: aclDetailsSlice #251
Conversation
Turns out even more of the parameters for the addNotification function in the notificationThunks are perfectly optional.
Fixes a "can't access lexical declaration 'X' before initialization" error, by resolving a kind of circular dependency.
Switching to redux toolkit for getting details for the acl modal.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
acl stuff also appears to be working. (See #242 (review) for more info on these approvals)
edit: whoops this should be an approval.
This pull request has conflicts ☹ |
This pull request has conflicts ☹ |
Helps with #213.
Switching to redux toolkit for getting details for the acl modal.