Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Lesson alert notifications #113

Merged
merged 15 commits into from
Nov 6, 2023
Merged

Lesson alert notifications #113

merged 15 commits into from
Nov 6, 2023

Conversation

joaoalves03
Copy link
Collaborator

@joaoalves03 joaoalves03 commented Nov 6, 2023 •
edited
Loading

Adds alert notifications X minutes before a lesson starts, and also lays the groundwork for lesson changes notifications #112

image
image
image

joaoalves03 and others added 14 commits October 22, 2023 13:15
@joaoalves03
0.23.0
eb34107
@joaoalves03
[WIP] Lesson alert notifications
99cb682
@joaoalves03
Lesson alerts are now only scheduled for the next 24 hours
8ce431e
@joaoalves03
Fixed alerts only being scheduled 5m before
1992c1c
@joaoalves03
Added icon for notifications on android
80dc089
@joaoalves03
Fetching schedule function no longer schedules alerts automatically
0c4f84f
@joaoalves03
Added a background task to schedule notifications every 24h
8d1d71d
@joaoalves03
Disabled background alert scheduling if it's disabled
4f78852
@joaoalves03
Fixed notification icon on android
f5839e3
@joaoalves03
Verify if notification permission is granted before scheduling notifi…
f1032aa 
…cations
@joaoalves03
Check if lessonAlert setting is disabled before parsing schedule for …
b13b1df 
…notifications
@joaoalves03
Increased notification priority to max
678f266
@joaoalves03
Small fix number 32787832
4e3a9c0
@joaoalves03
Added leading icon to lesson alert setting
721f02a
sousa-andre
sousa-andre reviewed Nov 6, 2023
View reviewed changes
lib/providers/settings_provider.dart Outdated Show resolved Hide resolved
lib/services/background_actions.dart Outdated Show resolved Hide resolved
lib/ui/views/settings/notifications.dart Outdated Show resolved Hide resolved
@joaoalves03 joaoalves03 merged commit 51115b8 into dev Nov 6, 2023
@joaoalves03 joaoalves03 deleted the 10-add-notifications branch November 6, 2023 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sousa-andre sousa-andre sousa-andre left review comments

@rodrigosa7 rodrigosa7 Awaiting requested review from rodrigosa7

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@joaoalves03 @sousa-andre