-
Notifications
You must be signed in to change notification settings - Fork 2
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
Docs on cache busting #97
base: master
Are you sure you want to change the base?
Conversation
Notifying user smnalex (Slack username alexandru.simion) on Slack of PR request |
bff1fd3
to
be374c6
Compare
cache.bust(url) | ||
``` | ||
|
||
Note that the URL has to match exactly. For example, `/` suffix would need to be busted separately. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this mean we'd have to do both
/rider_applications/create_fountain_application
AND
rider_applications/create_fountain_application
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, it means you might need https://rider-domain.deliveroo.net/
as well as https://rider-domain.deliveroo.net
depending on what your code does. Unlikely though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.