8000 Add initial value to `DateTimePicker` and load existing value by quietpigeon · Pull Request #817 · igorescodro/alkaa · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add initial value to DateTimePicker and load existing value #817

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

Merged

Conversation

quietpigeon
Copy link
Contributor

This commit fixes #769

  • DateTimePicker now has a field for its initial value
  • The alarm will load the existing value if it exists

@quietpigeon quietpigeon changed the title code: add initial value to DateTimePicker and load existing value if Add initial value to DateTimePicker and load existing value if Jan 13, 2025
@quietpigeon quietpigeon changed the title Add initial value to DateTimePicker and load existing value if Add initial value to DateTimePicker and load existing value Jan 13, 2025
@igorescodro igorescodro force-pushed the feat/load-existing-alarm-datetime branch from f2d2841 to 4e8a5df Compare January 15, 2025 13:59
@igorescodro
Copy link
Owner

Hello, @quietpigeon.

Thank you for your contribution to Alkaa! All the tests are passing and I validated locally and everything is working properly.
Some linter errors needs to be fixed before merging the pull request.

Let me know if you have time to take a look at them.

@igorescodro igorescodro merged commit 23d136a into igorescodro:main Jan 16, 2025
5 checks passed
@quietpigeon quietpigeon deleted the feat/load-existing-alarm-datetime branch February 15, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alarm date-time does not initially load on screen when editing
2 participants
0