Skip to content

Conversation

@niko-exo
Copy link

@niko-exo niko-exo commented Jan 30, 2026

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

image

Summary by CodeRabbit

  • Bug Fixes
    • Improved image loading in the sponsor inventory to prevent display of stale cached content.

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

A cache-busting query parameter is added to image filenames in the sponsor inventory popup form template. The modification appends a timestamp-based parameter to prevent browser caching of images during form interactions.

Changes

Cohort / File(s) Summary
Image Cache Busting
src/pages/sponsors-global/form-templates/sponsor-inventory-popup.js
Added timestamp-based cache-busting query parameter (?buster=[timestamp]) to image filenames in the media input value mapping to prevent stale image caching.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • smarcet
  • romanetar

Poem

🐰 Cache-busting tales we tell,
With timestamps worn like a bell,
No stale images stay,
Fresh pictures each day,
In the inventory carousel!

🚥 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

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants