-
Notifications
You must be signed in to change notification settings - Fork 170
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
Handle errors when updating changelog #617
Conversation
@marcellogalhardo Can you enable the option to allow maintainers to edit your pull request? We're ready to merge but we don't merge if the PR is out of date with main. If we can edit the branch I can get this merged tonight. |
@twizmwazin you should have the rights now! I'm currently in holidays, so I have limited access to internet and I didn't see your notification before. Sorry for the inconvenience. |
Mmmk I got this one updated, it should get merged soon. |
AppSettingsRepository
;scope.launch
byLaunchedEffect
to avoid unnecessary calls during recompositions;getValue
delegate on states to reduce verbosity (no need forstate.value =
each time);Fixes #378