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
First run changes for invite page #2221
Conversation
1fb0214
to
5e118e0
Compare
@@ -19,6 +19,7 @@ const defaultState = { | |||
showUnavailable: getLocalStorageSetting(SETTINGS.SHOW_UNAVAILABLE, true), | |||
welcome_acknowledged: getLocalStorageSetting(SETTINGS.NEW_USER_ACKNOWLEDGED, false), | |||
email_collection_acknowledged: getLocalStorageSetting(SETTINGS.EMAIL_COLLECTION_ACKNOWLEDGED), | |||
invite_acknowledged: getLocalStorageSetting(SETTINGS.INVITE_ACKNOWLEDGED, false), |
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.
All of these first run "settings" probably shouldn't live here. Not sure how tricky it would be to migrate them just to regular values saved by redux-persist. Then we can get rid of getLocalStorageSetting
.
Works fine for now though, but something to keep in mind.
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.
kill it with 🔥
label: 'Overview', | ||
path: '/wallet', | ||
active: currentPage === 'wallet', | ||
...buildLink('Overview', PAGES.WALLET), |
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.
Could make an array of these pairs and do a map.
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.
Ah doh. Good idea.
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.
Eh I think I'll leave it. Some of the links are slightly different, and then I would need to change all of those into objects, so there will still be some repetition.
Changes
Notes