-
Notifications
You must be signed in to change notification settings - Fork 903
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
Change activeKeyword in store to activeSections #9305
Labels
Comments
2 tasks
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, we're storing the activeKeyword in the redux state. However, when we're moving the content analysis to the sidebar, we'll have sections, where multiple sections can be active at the same time:
This means there will be no single active keyword anymore. Therefore, we should change the state to contain
activeSections
, which should be an array (instead ofactiveKeyword
, which is a string).Current state:
Future state:
Please branch from feature/content-analysis-in-sidebar
NB: Connecting to the UI is not part of the scope of the issue. This issue is purely about the actions and reducer.
The text was updated successfully, but these errors were encountered: