this post was submitted on 09 Apr 2025
16 points (94.4% liked)
Summit
932 readers
4 users here now
Community to discuss Summit, a Lemmy reader for Android.
App (Play Store): https://play.google.com/store/apps/details?id=com.idunnololz.summit
APK: https://github.com/idunnololz/summit-for-lemmy/releases
Source: https://github.com/idunnololz/summit
Support the app
Website: https://summit.idunnololz.com/
founded 2 years ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
Wouldn't it be more future proof and simple to return a boolean indicating whether the dialog should be shown or not, rather than delegating the calendar logic to every client?
That way different clients don't have to implement the "has a year passed" logic, and you retain the ability to change how often the dialog is shown too without requiring all clients to update.
Hmm that may be true, but now its already implemented like this, and this would require a breaking change. So better to leave it as is. Anyway the logic should not be changed.