Skip to content

feat: Add reminder feature - #3056

Draft
Elouan1411 wants to merge 131 commits into
mainfrom
protected/reminder
Draft

feat: Add reminder feature#3056
Elouan1411 wants to merge 131 commits into
mainfrom
protected/reminder

Conversation

@Elouan1411

@Elouan1411 Elouan1411 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@Elouan1411 @FabianDevel
feat: Add strings for new fragment
4d36805
@Elouan1411 @FabianDevel
feat: Add checkmark in view item setting
d26a4f3
@Elouan1411 @FabianDevel
feat: Add description in view setting radio button
7176406
@Elouan1411 @FabianDevel
feat: Add removeSubtitle and icon color in item setting view
445d5b2
@Elouan1411 @FabianDevel
feat: Add feature flag
0711e1f
@Elouan1411 @FabianDevel
feat: Add reminder/schedule strings and dimens
885d98e
@Elouan1411 @FabianDevel
feat: Add schedule/reminder config models and formatting helpers
609d42b
@Elouan1411 @FabianDevel
feat: Add scheduleConfig/reminderConfig state to NewMessageViewModel
06fe12f
@Elouan1411 @FabianDevel
feat: Replace compose schedule dialog with DraftSendOptionsFragment (…
e5cfb21 
…schedule + reminder options)
@Elouan1411 @FabianDevel
fix: Correct feature flag
4c57d45
@Elouan1411 @FabianDevel
feat: Add banner in newMessageFragment
e594bd5
@Elouan1411 @FabianDevel
fix: Correct click behavior on schedule option
8e72bce
@Elouan1411 @FabianDevel
fix: Correct display last schedule option
39c9a80
@Elouan1411 @FabianDevel
refactor: Remove ScheduleOptionsHelper to simplify the setup
c6cb07b
@Elouan1411 @FabianDevel
refactor: Add parameter names
43c595f
@Elouan1411 @FabianDevel
refactor: Clean code
4cd4491
@Elouan1411 @FabianDevel
refactor: Create handleCustomScheduleOptionClicked
34fe1dd
@Elouan1411 @FabianDevel
feat: Add annotations
b9e4474
@Elouan1411 @FabianDevel
fix: Remove divider to improve UI
e741a66
@Elouan1411 @FabianDevel
fix: Remove save button
f07ade5
@Elouan1411 @FabianDevel
feat: Use SimpleSettingView to have the same feel as the other screen…
413a0f7 
… in the app
@Elouan1411 @FabianDevel
refactor: Remove useless LinearLayout
a942bc5
@Elouan1411 @FabianDevel
refactor: Initialize constants in a single location
039012d
@Elouan1411 @FabianDevel
refactor: Convert liveData to stateFlow
bc3ba5b
@Elouan1411 @FabianDevel
refactor: Update UI with observer
2fa99ca
@Elouan1411 @FabianDevel
refactor: Rename functions and relocate logic to appropriate locations
ade50c6
@Elouan1411 @FabianDevel
refactor: Clean code
3d69870
@Elouan1411 @FabianDevel
refactor: Implement trailing content management for schedule options …
6a3ae86 
…and bottom sheets
@Elouan1411 @FabianDevel
fix: Correct matomo
e50d998
@Elouan1411 @FabianDevel
refactor: Remove unnecessary code
11196fe
tevincent and others added 26 commits August 28, 2026 15:07
@tevincent
feat: Add matomo (#3017)
48ddbab
@Elouan1411
feat: Add popup when the schedule is invalid
a80b347
@Elouan1411
feat: Add warning color for invalid schedule
e7a1a8f
@Elouan1411
feat: Update string
f7cecd4
@Elouan1411
Revert "feat: Add warning color for invalid schedule"
82d09f1 
This reverts commit a396f0b.

# Conflicts:
#	app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageFragment.kt
@Elouan1411
refactor: Clean code
47d6ebb 
feat: Update text displayed when the schedule is invalid
@Elouan1411
feat: Update text displayed when the schedule is invalid
92e5d6e
@Elouan1411
refactor: Clean code
0f39abe
@Elouan1411
feat: Add dialog
4d70095
@tevincent
fix: Handle expired schedules (#3043)
47088f4
@Elouan1411
feat: Add new toggle to change acknowledgement status for each mail
26acdea
@Elouan1411
feat: Add icon
c545eac
@tevincent
feat: Add new toggle to change acknowledgement status for each mail (#…
e698e87 
…3046)
@Elouan1411
feat: Sync reminder icon visibility across devices
fd66231
@Elouan1411
feat: Refresh message on click if there was an activity update with r…
87cc384 
…eminder
@Elouan1411
fix: Correct snapshot
bfdc912
@Elouan1411
chore: Bump MAILBOX_CONTENT_SCHEMA_VERSION
afc68e5
@Elouan1411
fix: Do not use isReminder which is specific to the hidden message
b6527f5
@Elouan1411
refactor: Rename shouldHideReminder
f010e6c
@Elouan1411
fix: Display Add reminder button if there are no messages at the end
3b958bf
@Elouan1411
fix: Refresh message when onExpanded
a3f081d
@tevincent
feat: Sync reminder icon visibility across devices and refresh messag…
acb821b 
…e when the reminder is added by the backend (#3051)
@FabianDevel
fix(Acknowledgment): Keep the general settings value in DraftSendOpti…
ef25d7c 
…onFragment
@tevincent
fix(Acknowledgment): Keep the general settings value in DraftSendOpti…
0e2593d 
…onFragment (#3054)
@Elouan1411
fix: Correct issue when saving a follow-up draft
42210e4
@tevincent
fix: Correct snapshot draft (#3055)
cbde100
@sonarqubecloud

sonarqubecloud Bot commented Aug 31, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
1 Accepted issue

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Elouan1411

Elouan1411 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

You need to display the preview of the original message, not the hidden message (in the thread list)

@Elouan1411

Elouan1411 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

There is also the fact of using only a single variable instead of both displayReminder and isReminder, since these two variables are always used together in our code for one specific case. Therefore, either the backend will change this to return a single variable, or there is likely a concept that was not understood during the feature's development, which will surely require a small modification on the mobile app side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot will automatically review once the pull request is marked ready for review

At least 1 approving review is required to merge this pull request.

Assignees

@Elouan1411 Elouan1411

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Elouan1411 @tevincent @FabianDevel