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

[bugfix] fix emoji recaching operations #3167

Merged
merged 14 commits into from
Aug 3, 2024
Merged

[bugfix] fix emoji recaching operations #3167

merged 14 commits into from
Aug 3, 2024

Conversation

NyaaaWhatsUpDoc
Copy link
Member

@NyaaaWhatsUpDoc NyaaaWhatsUpDoc commented Aug 3, 2024 •
edited
Loading

  • update the recache operation in internal/media to actually include the instance account ID, which it previously wasn't doing
  • update the refresh emoji operation in the dereferencer to not perform a full refresh in the case that it wasn't updated but it's just not cached

@NyaaaWhatsUpDoc NyaaaWhatsUpDoc changed the title [bugfix] update emoji recache to set the instance account id [bugfix] fix emoji recaching operations Aug 3, 2024
@tsmethurst tsmethurst merged commit b85a998 into superseriousbusiness:main Aug 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NyaaaWhatsUpDoc @tsmethurst