Skip to content

[current_time] Initial commit for skill to say current time#91

Merged
Stypox merged 4 commits intoStypox:masterfrom
drew-sinha:current_time
Jul 26, 2022
Merged

[current_time] Initial commit for skill to say current time#91
Stypox merged 4 commits intoStypox:masterfrom
drew-sinha:current_time

Conversation

@drew-sinha
Copy link
Contributor

@drew-sinha drew-sinha commented Jul 17, 2022

This PR provides a new feature/skill allowing the user to poll dicio for the current time. This skill is added to the en/default locale but not updated to any others. This skill has been verified working in debug builds using OpenJDK 14/Gradle 7.4 installed on Android 12.

(BTW, thanks for providing this infrastructure. Has been a good way to get into the Android app environment/ecosystem, and dicio is a good asset to the community.)

Edit 7/23: This skill now uses its own material design logo.

Copy link
Owner

@Stypox Stypox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Code mostly looks good, my comments are just some nitpicking. I am glad you found the documentation and the ecosystem understandable :-)

@drew-sinha
Copy link
Contributor Author

All changes implemented.

Copy link
Owner

@Stypox Stypox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! I applied some slight modifications in 0fe8973 and translated to Italian in ce0c9f6 :-)

@Stypox Stypox merged commit 26d9a3d into Stypox:master Jul 26, 2022
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.

2 participants