Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CommentRepliesSource: Add cache for reply fetching #11718

Open
Profpatsch opened this issue Nov 18, 2024 · 2 comments
Open

CommentRepliesSource: Add cache for reply fetching #11718

Profpatsch opened this issue Nov 18, 2024 · 2 comments
Labels
bug Issue is related to a bug comments Anything to do with comments and comment replies under videos/audios rewrite Issues and PRs related to rewrite

Comments

@Profpatsch
Copy link
Contributor

There is no cache in the comment replies fetching, causing the app to reload them each time the device is rotated or a switch between the description content view tab and the comments one is done. This regression must be fixed.

Originally posted by @AudricV in #11060 (comment)
@Profpatsch Profpatsch moved this to Todo in Rewrite Nov 18, 2024
@AudricV AudricV added bug Issue is related to a bug comments Anything to do with comments and comment replies under videos/audios rewrite Issues and PRs related to rewrite labels Nov 18, 2024
@SttApollo
Copy link

Can you assign this to me please?

@opusforlife2
Copy link
Collaborator

I've assigned you to another issue for now. It would be better to tackle things one at a time. Once you get used to what the devs expect in terms of code, you're welcome to take up further issues.

@opusforlife2 opusforlife2 mentioned this issue Nov 24, 2024
Open
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Issue is related to a bug comments Anything to do with comments and comment replies under videos/audios rewrite Issues and PRs related to rewrite
Projects
Rewrite
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Profpatsch @opusforlife2 @AudricV @SttApollo