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
refactor: Use useSettings
hook in DefaultRedirectionSnackbar
#2117
base: master
Are you sure you want to change the base?
Conversation
...homeSettings, | ||
default_redirection_view_count: default_redirection_view_count | ||
? default_redirection_view_count + 1 | ||
: 1 |
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.
I do not see anymore this logic of setting 1 if there was no value before. Will it work ?
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.
Fixed here and in cozy-client
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.
Finally we did not include the default parameter option in the useSettings API, so I added the necessary code in this cozy-home PR
f17fe1a
to
7070fa5
Compare
7814cd4
to
7542e40
Compare
`cozy-client` has been upgraded to `46.9.0` to retrieve the new useSettings hook Related PR: cozy/cozy-client#1460
In cozy/cozy-client#1460 we implemented a new `useSettings` hook This hook can be used in `DefaultRedirectionSnackbar` in order to simplify code and homogenize the way we access cozy-apps settings
useSettings
hook in DefaultRedirectionSnackbar
7542e40
to
349b630
Compare
BundleMonFiles updated (7)
Unchanged files (6)
Total files change -80.05KB -2.55% Final result: β View report in BundleMon website β‘οΈ |
In cozy/cozy-client#1460 we implemented a new
useSettings
hookThis hook can be used in
DefaultRedirectionSnackbar
in order to simplify code and homogenize the way we access cozy-apps settings