this post was submitted on 10 Mar 2024
44 points (97.8% liked)

Summit

744 readers
11 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

Patreon: https://patreon.com/SummitforLemmy

Ko-Fi: https://ko-fi.com/summitforlemmy

Website: https://summit.idunnololz.com/

founded 1 year ago
MODERATORS
 

This update adds notifications support to the app. Notifications will need to be enabled on a per-account basis to reduce the amount of unnecessary API calls. You can find notifications & their per-account settings in Settings > Notifications. By default notifications are disabled.

When notifications are enabled, you will get notified of any unread DMs, mentions or replies to your posts. In addition, if you are an admin or mod, you will also get notified of reports.

Like previous updates, this update also adds a lot of quality of life improvements such as better video support and bug fixes. See the full change log for details.

Due to notifications being a big change, this release will be rolled out slowly over the next two days to reduce the impact of bugs on users.

Full change log

  • Added notifications as an optional feature to the app. This feature is off by default. To enable, go to Settings > Notifications.
  • Added support for RedGifs.
  • Added more options in the built-in video player.
  • Changed the comment screen to auto focus the input field when first shown.
  • Fixed an issue where custom comment gesture actions where not being applied.
  • Fixed a regression with screenshot mode where post actions are not hidden when in screenshot mode.
  • Fixed some inconsistent scrolling behavior throughout the app.

Update

Rollout complete. There is one minor bug found but it was very minor so I will punt for next update. The bug is, if you tap on a notification, it will take you to the message in your inbox, however when you go back (to your inbox) the message looks like its not read yet. This is a client side bug only; the message is correctly marked as read on the server.

Update 2

There is a very minor update for the Github release to fix a crash that is only present in the Github release (there was some code that had undefined behavior so the Github and PlayStore versions had different behavior despite being compiled from the same code). Thus the Github release is v1.29.1.

all 18 comments
sorted by: hot top controversial new old
[–] ZebraGoose@sh.itjust.works 7 points 8 months ago (1 children)
[–] idunnololz@lemmy.world 4 points 8 months ago
[–] shnizmuffin@lemmy.inbutts.lol 6 points 8 months ago
[–] WeeScottishMan@sh.itjust.works 4 points 8 months ago

Keep up the amazing work, it's a pleasure to be able to use this app everyday and it just keeps getting better too😊✨

[–] LaurenceWolse@feddit.nl 3 points 8 months ago (4 children)

I am experiencing a small bug since this last update. I have the user profile button added to my nav bar. Pushing it crashes the app. The profile is still reachable without crashing by pushing the avatar in the top right corner.

Thanks for adding notification support! It rocks! :)

[–] idunnololz@lemmy.world 3 points 8 months ago (1 children)

Fix should be out on GitHub. Please try it out and let me know if the issue has been fixed. Thanks!

[–] LaurenceWolse@feddit.nl 3 points 8 months ago

The issue has indeed been fixed. Thanks for your hard work!

[–] idunnololz@lemmy.world 2 points 8 months ago (1 children)

I can't reproduce this myself but I will take a look later today.

[–] idunnololz@lemmy.world 2 points 8 months ago

Took a peak at crash logs and I think I found yours. The only issue is that technically the crash mentioned in the logs is impossible or at least it doesn't make sense given the code... Which is a problem. It does explain why I can't reproduce it. This might be a bit involved to debug.

[–] idunnololz@lemmy.world 2 points 8 months ago* (last edited 8 months ago)

One other question I had. From where did you install the update? The play store or the github page?

[–] idunnololz@lemmy.world 2 points 8 months ago (1 children)

Could you send me your settings configuration? This will help me debug the issue as the crash does appear to occur under normal circumstances. You can get it by going to Settings > Import & export settings > Export settings. Then tap copy to clipboard. You can send it to me here or DM me.

[–] LaurenceWolse@feddit.nl 2 points 8 months ago (1 children)
[–] idunnololz@lemmy.world 2 points 8 months ago

Ah actually I finally got a reproduction. It turns out, parts of my code has undefined behavior. So compilation of the code in these part is not well defined. Eg. each time the code is compiled it can do A or B and both are technically correct interpretations of the code.

This means that I can change nothing about my code and technically we can see one release do A and another do B. In this case, the play store release is actually fine but the github release is not. In addition, the previous versions are fine and this release isn't even though the logic didn't change. I have a fix planned and I will do a release soon. Since this issue only affects the github release, I do not need to do a release for the play store version.