Issue 1556: global options to hide reposts and replies#1771
Issue 1556: global options to hide reposts and replies#1771danguyf wants to merge 18 commits intoDimensionDev:masterfrom
Conversation
Tlaster
left a comment
There was a problem hiding this comment.
I'm keeping #1556 in the backlog because I'm still considering the best way to fix the issue.
The easiest fix would be to filter out all replies in dev.dimension.flare.data.datasource.bluesky.HomeTimelineRemoteMediator before saving them to the database, align with other social platforms. However, that would prevent others who want to see the replies in their timeline from doing so. Therefore, this is not the right fix.
This PR updates the TimelinePresenter, meaning the entire application will apply the filter. This will also filter out reposts and replies in other scenarios where the user does not want to apply the filter, such as in profile timeline. This makes future improvements to the user's profile timeline much harder. For example, it would be difficult to add a filter to the user's profile timeline to filter out reposts, replies, and media on the profile page. This filter would not apply globally. I just want to see what the user posts that does not contain reposts, replies, and media.
The closest thing I can think of is making this configuration per home timeline tab (this is also what OpenVibe is doing), which depends on #1388.
compose-ui/src/commonMain/kotlin/dev/dimension/flare/data/model/AppearanceSettings.kt
Outdated
Show resolved
Hide resolved
compose-ui/src/commonMain/kotlin/dev/dimension/flare/data/repository/SettingsRepository.kt
Outdated
Show resolved
Hide resolved
compose-ui/src/commonMain/kotlin/dev/dimension/flare/di/ComposeUiModule.kt
Outdated
Show resolved
Hide resolved
shared/src/commonMain/kotlin/dev/dimension/flare/ui/presenter/home/TimelinePresenter.kt
Show resolved
Hide resolved
shared/src/commonMain/kotlin/dev/dimension/flare/ui/presenter/home/TimelinePresenter.kt
Outdated
Show resolved
Hide resolved
|
These are all good notes. I realized the issue with hiding replies on the user page at one point and fixed that but apparently it was not in this branch -- ack! If I port it over and address your other points might this be worth considering? |
|
Thanks for the follow up, but I’m afraid this won’t work as-is. |
…k file through and I never notice.
|
These options are now Home Feed only and stripped of intermediary cruft. |
|
Maybe I wasn’t clear earlier. |
|
Okay! This was a good learning experience. I look forward to #1388 and potential future features stemming from it. |
Issue: #1556
My ultimate goal is to implement the feature requested in #1711 and this issue seemed like a good, simpler use case in order to get familiar with how reposts and replies are identified and displayed so I started here.