-
Notifications
You must be signed in to change notification settings - Fork 163
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
Simplifies API usage #405
Merged
Merged
Simplifies API usage #405
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- update getDate, getTime API
Emulator is working ! |
Closed
RuffaloLavoisier
added
⭐ enhancement
New feature or request
src/osw-app
This PR / issue impacts the current release of the OS
src/core
src/osw-watchface
labels
Sep 29, 2024
Some thoughts of mine:
|
simonmicro
requested changes
Sep 29, 2024
- init value, method proto type, structure style
RuffaloLavoisier
force-pushed
the
feature/DateTimeAPI
branch
from
October 7, 2024 12:14
7312ae2
to
5b8d3dd
Compare
RuffaloLavoisier
force-pushed
the
feature/DateTimeAPI
branch
from
October 7, 2024 12:26
5f57b54
to
d8fb1aa
Compare
simonmicro
requested changes
Oct 14, 2024
Merged
1 task
simonmicro
reviewed
Oct 17, 2024
simonmicro
approved these changes
Oct 18, 2024
Hey, the changes to the code itself look fine - but, as expected, LUA is still not compatible with the deprecation attribute. I would recommend you to revert that commit for now and move it into #366, as that PR also aims to fix that. |
RuffaloLavoisier
added a commit
that referenced
this pull request
Oct 18, 2024
RuffaloLavoisier
added a commit
that referenced
this pull request
Oct 18, 2024
RuffaloLavoisier
force-pushed
the
feature/DateTimeAPI
branch
from
October 18, 2024 11:05
9bb1d31
to
586b004
Compare
- Gets the name of the day of the week that the cursor points to
@simonmicro Ready to merge. |
simonmicro
approved these changes
Oct 22, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
src/core
src/osw-app
This PR / issue impacts the current release of the OS
src/osw-watchface
⭐ enhancement
New feature or request
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update getDate, getTime API with #403
How to use
Each element can be referred to as a sub-element. It returns data for a time or date and can be used by selecting the necessary elements.
Summary
internal
getUTCTime
APIto
internal
getTime
APIto
merge internal
getDate
APIto
update
getLocalTime
APIto
merge
getLocalDate
APIto
update
getDualTime
APIto
merge
getDualDate
APIto
update
getWeekDay
API