-
Notifications
You must be signed in to change notification settings - Fork 1
Feat/notifications #1386
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/notifications #1386
Conversation
…ive methode using shared preferences
|
Label enhancement 🚀 has been added to this pull request |
2006d8d to
dae4265
Compare
|
Label enhancement 🚀 has been added to this pull request |
dae4265 to
b488b17
Compare
|
Label enhancement 🚀 has been added to this pull request |
b488b17 to
96a6c3c
Compare
|
Label enhancement 🚀 has been added to this pull request |
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.
Please translate hard coded text on Android part
What is the content of the IOS notification ? Where is the text ?
yaki_mobile/android/app/src/main/kotlin/com/xpeho/yaki/AlarmReceiver.kt
Outdated
Show resolved
Hide resolved
yaki_mobile/android/app/src/main/kotlin/com/xpeho/yaki/AlarmReceiver.kt
Outdated
Show resolved
Hide resolved
yaki_mobile/android/app/src/main/kotlin/com/xpeho/yaki/AlarmSchedulerImpl.kt
Outdated
Show resolved
Hide resolved
yaki_mobile/android/app/src/main/kotlin/com/xpeho/yaki/MainActivity.kt
Outdated
Show resolved
Hide resolved
yaki_mobile/android/app/src/main/kotlin/com/xpeho/yaki/MainActivity.kt
Outdated
Show resolved
Hide resolved
96a6c3c to
a9200de
Compare
|
Label enhancement 🚀 has been added to this pull request |
a9200de to
ce1e61f
Compare
|
Label enhancement 🚀 has been added to this pull request |
ce1e61f to
0143e71
Compare
|
Label enhancement 🚀 has been added to this pull request |
0143e71 to
a1e824f
Compare
|
Label enhancement 🚀 has been added to this pull request |
a1e824f to
21c6a3e
Compare
|
Label enhancement 🚀 has been added to this pull request |
21c6a3e to
240c4b8
Compare
|
Label enhancement 🚀 has been added to this pull request |
240c4b8 to
9b8ed20
Compare
|
Label enhancement 🚀 has been added to this pull request |
9b8ed20 to
c7f11b5
Compare
|
Label enhancement 🚀 has been added to this pull request |
c7f11b5 to
e820603
Compare
|
Label enhancement 🚀 has been added to this pull request |
e820603 to
3086854
Compare
|
Label enhancement 🚀 has been added to this pull request |
3086854 to
abede6f
Compare
|
Label enhancement 🚀 has been added to this pull request |
abede6f to
9e73fac
Compare
|
Label enhancement 🚀 has been added to this pull request |
9e73fac to
045dd01
Compare
|
Label enhancement 🚀 has been added to this pull request |
045dd01 to
fb8ebe1
Compare
|
Label enhancement 🚀 has been added to this pull request |
fb8ebe1 to
8938551
Compare
|
Label enhancement 🚀 has been added to this pull request |
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.
LGTM
Description
What are changes related to ?
We have previously added a daily reminder by native code:
feat(android_notif): Add daily notification in android platform and set the option in profile view #1380
feat(IOS_notification): create IOS local notification, and flutter channel to enable or disable them #1381
Here we improve the work on these two pull requests and add the ability to enable or disable the notification in the profile view.
We also make different calls from native to flutter and from flutter to native to ensure consistency of each data item regarding activation or permission to send a notification.
Tests
How has it been tested ?