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

Investigate caching. #5

Open
khipkin opened this issue Feb 26, 2015 · 6 comments
Open

Investigate caching. #5

khipkin opened this issue Feb 26, 2015 · 6 comments

Comments

@khipkin
Copy link
Member

khipkin commented Feb 26, 2015

Right now, loading our calendar takes a significant amount of time. It seems silly to reload the entire thing every time a user navigates to the page.

@chrissprague
Copy link
Member

@khipkin can we make this issue a bit more specific? Like, would this be caching in general or are we just looking to do this for the calendar?

@khipkin
Copy link
Member Author

khipkin commented Feb 27, 2015

@chrissprague, not sure. It depends how easy it is to implement.

@zg
Copy link
Member

zg commented May 14, 2015

This could be done either in Django or Apache. It'd probably be easier to do it at the HTTP server layer (Apache).

@zg
Copy link
Member

zg commented May 19, 2016

Wait, don't we just use Google Calendar? Were we using something else when you posted this?

@khipkin
Copy link
Member Author

khipkin commented May 19, 2016

If you mean the Google Calendar API, no we don't. We couldn't figure out how to properly authenticate to use it. Also, caching pictures would be useful, although that might happen automatically depending on the browser.

@zg
Copy link
Member

zg commented Sep 7, 2016

image

I ran an Audit using Chrome Dev Tools and they suggest leveraging browser caching for all the images, CSS, and JS files. I think this would have to get changed in the Apache config.

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

No branches or pull requests

3 participants