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

BulkUpload FormsManager references stale thumbnails after partial success #10650

Open
akhrarovsaid opened this issue Jan 17, 2025 · 0 comments · May be fixed by #10651
Open

BulkUpload FormsManager references stale thumbnails after partial success #10650

akhrarovsaid opened this issue Jan 17, 2025 · 0 comments · May be fixed by #10651
Labels
created-by: Contributor status: needs-triage Possible bug which hasn't been reproduced yet

Comments

@akhrarovsaid
Copy link
Contributor

akhrarovsaid commented Jan 17, 2025

Describe the Bug

The thumbnails that are generated in the bulk upload drawer do not get removed for successful saves. Say, for example, if you are uploading two images and each image requires an alt field to be populated, and only one image passes validation, the remaining image will have the stale thumbnail of the first entry visible on it.

Observe this issue here:
https://github.com/user-attachments/assets/b8878844-dd0a-4f24-986c-25c6d0e31233

Link to the code that reproduces this issue

https://github.com/akhrarovsaid/payload/tree/repro/bulkupload-stale-thumbnail

Reproduction Steps

  • Grab repro
  • Navigate to posts
  • Click create new to trigger bulk upload drawer to open
  • Add 2 images (it's important that these files are images so that we generate thumbnails for them)
  • Add an alt to only the first image and submit
  • Observe remaining image with stale thumbnail of previous image

Which area(s) are affected? (Select all that apply)

area: ui

Environment Info

Payload: 3.17.1+
Node: 20+
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
created-by: Contributor status: needs-triage Possible bug which hasn't been reproduced yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant