-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat: use custom component for time of day setting #53
base: main
Are you sure you want to change the base?
Conversation
fbf7097
to
26522d3
Compare
26522d3
to
4a09178
Compare
Hey @mickmister, this is not finished yet, but can you check if what I did in the frontend seems correct? It's working but the console is complaining about a value not being the type is supposed to be with an error. I based it in the work you did with S3. |
@fmartingr I'm not seeing this error you're mentioning. Can you post the text or screenshot of the error? |
Mhmmm that's weird. |
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.
Approach looks good to me in general.
/update-branch |
No description provided.