Summit

1277 readers

25 users here now

Community to discuss Summit, an open-source 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

idunnololz@lemmy.world

[Release][v1.80.5] PieFed mod tools and performance improvements (lemmy.world)

submitted 1 week ago* (last edited 5 days ago) by idunnololz@lemmy.world to c/summit@lemmy.world

25 comments fedilink hide all child comments

This release adds some mod tools for PieFed to the app (some mod tools are still missing because they are not supported by the API yet).

This release also makes significant changes under the hood. These changes are meant to keep the app snappy. The app goes through lots of updates and features are added left and right so some occasional refactoring is necessary to ensure the app does not get too bloated.

Changes to the internal portions of the app is also why this release took a bit longer. These changes can cause bugs if not done correctly and I had to dogfood the app for a while before I was confident enough to release the update.

Full changelog

Add support for modlogs on piefed.
Add support for reviewing and accepting/rejecting registrations.
Add support for [spoiler] tags.
Fix some bugs with finding text on the post screen.
Optimized save/restore post feed when app is killed and restored. This adds a slightly longer delay when the app is restored after it's been killed by the OS but it should reduce the number of network calls and improve snappiness once the app is restored.
Show HD image toggle in more situations.
Fix a bug where the loading spinner never disappears on a post even though internally the post has failed to load.
Fix a bug where "No comments" is not shown when there are no comments.

Update v1.80.1

Fix a bug where long tapping on an image will load the thumbnail and not the original image.

Update v1.80.2

Fix a bug where sometimes resuming the app, the post feed will fail to restore.

Update v1.80.3

Fix a bug where link posts with a thumbnail incorrectly try to load the link and not the thumbnail.
Fix a bug where community icons sometimes flicker.
Fix a bug where viewing an image of a post will no longer default to the thumbnail if it has been cached.
Reverted change where opening a post from the post screen will open it in the same post window. This was causing confusing back navigation problems.

Update v1.80.4 I accidentally didn't build the correct code because the code was written on another computer and I also accidentally shipped some test code. Doing another release to correct these.

Update v1.80.5

There was a one line code change that I missed when cleaning up after experimening with something that was increasing crash rates. My bad.

you are viewing a single comment's thread
view the rest of the comments

[–] idunnololz@lemmy.world 10 points 1 week ago* (last edited 1 week ago)

Sorry the note is a bit misleading. The change that was made just happens to do both of these things.

The low level detail is that app caching logic was moved into a lower layer. Some caching logic was moved to the layer that was doing the network calls to fetch posts.

When the app restarts, that layer restores the cache so it can rely on the cache for some of the network calls. This is why there's less network calls.

Because the cache is at a lower level, more work has to be done to process the data which is why its slower. I can also do like a multilayer caching solution but I didn't think it was necessary.