-
Notifications
You must be signed in to change notification settings - Fork 32
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
Me/record field update frequency #868
Conversation
Affected libs:
|
📷 Screenshots are here! |
a251c8e
to
9c46cf6
Compare
9c46cf6
to
f93c34b
Compare
@jahow Unless we can save floats as duration in ISO format, I don't see how to solve the "3 times a week" => "every 2 days" => "4 times a week" issue (see last wip commit). |
ce3ecaf
to
15b62fc
Compare
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.
Thank you @LHBruneton-C2C , great work and easy review ;)
I've rewritten the last commit to make the converter stable when doing back and on forth on the update frequency field for frequencies higher than one week. Let me know if that makes sense to you! feel free to merge when ready.
Thanks @jahow for the last commit! |
Description
This PR introduces support of the update frequency field in the record form.
The dropdown selector can now be disabled.
Screenshots
Quality Assurance Checklist
breaking change
labelbackport <release branch>
label