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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEW] Setting to set a JS/CSS CDN #11779

Merged
merged 8 commits into from Aug 17, 2018
Merged

[NEW] Setting to set a JS/CSS CDN #11779

merged 8 commits into from Aug 17, 2018

Conversation

geekgonecrazy
Copy link
Member

@geekgonecrazy geekgonecrazy commented Aug 14, 2018

Adds a setting to use a CDN just for JS/CSS assets aka the ones bundled.

The existing setting CDN_PREFIX applies across js,css,avatars,emoji and everything.

The new setting CDN_PREFIX_ALL is set to true by default to preserve that existing behaviour.

But if you want to serve js/css from a different CDN prefix. Then you can set the new setting: CDN_JSCSS_PREFIX and it will then only apply the prefix provided to the js/css files. In our case.. all of the meteor bundle js/css files 馃槃

This will allow someone to start off and accelerate the js / css delivery speeding up initial paint. Then later on if they wish.. they can start using a CDN to cache more of their assets. This also gives them the power to combine as one CDN... or potentially use a couple of CDN's.

Power in the users hand. 馃槃

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11779 August 14, 2018 19:57 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11779 August 14, 2018 20:17 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11779 August 14, 2018 20:18 Inactive
@geekgonecrazy geekgonecrazy added this to the 0.69.0 milestone Aug 14, 2018
@geekgonecrazy geekgonecrazy added this to In progress in August/2018 via automation Aug 14, 2018
@RocketChat RocketChat deleted a comment Aug 14, 2018
@geekgonecrazy geekgonecrazy changed the title [NEW] Setting for a JS/CSS CDN [NEW] Setting to set a JS/CSS CDN Aug 14, 2018
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11779 August 17, 2018 00:57 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11779 August 17, 2018 01:03 Inactive
@RocketChat RocketChat deleted a comment Aug 17, 2018
@RocketChat RocketChat deleted a comment Aug 17, 2018
@RocketChat RocketChat deleted a comment Aug 17, 2018
@RocketChat RocketChat deleted a comment Aug 17, 2018
@RocketChat RocketChat deleted a comment Aug 17, 2018
August/2018 automation moved this from In progress to Ready to merge Aug 17, 2018
@sampaiodiego sampaiodiego merged commit 767b3c9 into develop Aug 17, 2018
August/2018 automation moved this from Ready to merge to Done Aug 17, 2018
@sampaiodiego sampaiodiego deleted the add/jscss_cdn branch August 17, 2018 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
August/2018
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants