-
Notifications
You must be signed in to change notification settings - Fork 24
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
Cache opsdroid icon for chat demo #14
Comments
This is definitely something that should be fixed. Thanks! |
I should add that I experience this lack of cache in Safari on High Sierra (will get specific numbers later) but not on Google Chrome Canary 57.0.2985.0 on Windows 7. Not sure, but I feel it’s related to how Safari determines cache and how Jekyll server reports what can be cached. |
I want to do my 1st contribution. Can I work on this issue? |
hey @jacobtomlinson I would like to work on this issue, is it still open @pjbollinger ? |
@himansh-gjr Feel free to open a PR with this changes as this is still open, thank you |
Sure, Thanks ! |
While working on adding blog posts to the site, I noticed a network request every time the opsdroid icon appeared in the chat demo. Ideally, this should be stored on the local machine at first load so the request never has to be completed again, until the page reloads.
The text was updated successfully, but these errors were encountered: