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.

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

Changing book metadata does not change post if user has 'finished reading' book #3510

Open
meowsoni opened this issue Feb 19, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@meowsoni
Copy link

Describe the bug
If you import a book with incorrect data (such as title or author) and click 'Read' immediately, thus creating a public post that says "[user] finished reading [wrong title]" then correcting the metadata afterward does not correct the line in the post.

To Reproduce
Steps to reproduce the behavior:

  1. Search for a book with incorrect details in Title (as in the attached screenshot) and Import the book from OpenLibrary.
  2. 'Read' the book so as to create a public post that goes "[user] finished reading [wrong title]".
  3. Change the title of the book manually and save.
  4. Go back to the post to find the incorrect metadata is still imprinted in the post and even the URL in the a href value reflects the incorrect name (though it seems to redirect to the updated URL).

Expected behavior
Editing the book should update the erroenous titles everywhere it appears. I wonder if the line "[user] finished reading [...] by [...]" imports static values for title and author at the time of post creation (is there a way to update this en-masse when a book's metadata is updated)

Screenshots
Screenshot attached

Instance
Proust.One

Steps taken already to fix
I have tried refreshing redis activity streams

meowsoni

Image

@meowsoni meowsoni added the bug Something isn't working label Feb 19, 2025
@meowsoni
Copy link
Author

I found a fix!

I first cleared the Django cache

docker-compose exec web python manage.py shell

Then:

from django.core.cache import cache
cache.clear()
exit()

And then I repopulated the streams using bw-dev populate_streams

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant