86b85g4ct - fix: add buster number to image url to avoid cache issues

[ghost]

ref: https://app.clickup.com/t/86b85g4ct

86b85g4ct - fix: add buster number to image url to avoid cache issues

Changelog

• Add buster number to image url to avoid cache issues

86b85g4ct - Showing wrong images in Item>Edit

Evidence

Summary by CodeRabbit

• Bug Fixes
  • Sponsor inventory images now include a timestamp parameter so updated images appear immediately in the sponsor inventory popup, avoiding stale cached thumbnails and improving visual consistency.
  • Minor display reliability improvements when opening the sponsor inventory form to ensure current media is shown.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Appends a timestamp-based cache-busting query parameter (?t=[timestamp]) to image filenames in the sponsor inventory popup media input mapping; data shape and control flow otherwise unchanged.

Changes

Cohort / File(s)	Summary
Image cache busting src/pages/sponsors-global/form-templates/sponsor-inventory-popup.js	Within the getMediaInputValue mapping for initialEntity.images, compute the image filename as before and append ?t=[timestamp] to the filename field returned by the map to force cache-busting. Minor local-variable and return-object changes only.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• 86b85g4ct - fix: add buster number to image url to avoid cache issues #765 — Makes the same change to the getMediaInputValue mapping in sponsor-inventory-popup.js, but appends a ?buster= timestamp parameter instead of ?t=.

Suggested reviewers

• romanetar

Poem

🐰
I hop through code with a twitch and a grin,
Appending a timestamp to keep images thin,
No stale sprites in sight,
Each load fresh and bright,
A bunny-approved cache-cleansing spin! 🥕✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main change: adding a cache-busting query parameter to image URLs to avoid caching issues, which matches the code modification that appends a timestamp-based buster parameter to image filenames.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/showing_wrong_images_in_item_edit

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[smarcet]

LGTM