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

Add quick cache buster to Bolt docs site #941

Merged
merged 2 commits into from
Nov 7, 2018

Conversation

sghoweri
Copy link
Contributor

@sghoweri sghoweri commented Nov 2, 2018

Jira

N/A

Summary

Adds a quick cache buster query string to the Bolt docs site head and footer’s main CSS and JS bundles.

Details

Quick workaround to the now.sh cache not always getting cleared in certain situations — namely when aliasing an existing deployment (like taking the existing deployment that’s running on the master.boltdesignsystem.com URL alias) and remapping it to another domain (like the live boltdesignsystem.com site).

Screenshots below show some of the old CSS showing up in the new version selector UI (before manually clearing the browser cache):

8b2dff4d-a393-461c-9e03-bb0d334b2f23
5d562407-bc28-45b3-9938-a749b3610ba6

Note: we already deal with cache busting hashes in async loaded bundles — this is specifically dealing with cache busting the main entry file loaded till we get a more advanced Webpack solution wired up for this.

How to test

Confirm docs site JS and CSS loading as expected.

@bolt-bot
Copy link
Collaborator

bolt-bot commented Nov 2, 2018

⚡ PR built on Travis and deployed a now preview here:

@bolt-bot
Copy link
Collaborator

bolt-bot commented Nov 2, 2018

⚡ PR built on Travis and deployed a now preview here:

@sghoweri sghoweri requested a review from remydenton November 6, 2018 21:06
Copy link
Collaborator

@remydenton remydenton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, thanks!

@remydenton remydenton merged commit 606d40d into master Nov 7, 2018
@remydenton remydenton deleted the feature/add-cache-buster branch November 7, 2018 16:56
@remydenton remydenton mentioned this pull request Nov 7, 2018
This was referenced Nov 18, 2018
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.

3 participants