Move OC.AppConfig, OC.EventSource and OC.L10N to the server bundle

[ChristophWurst]

And another three requests less per page load 🚀

[skjnldsv]

god I'm loving this <3
It's so beautiful I could cry 👀

[ChristophWurst]

❤️ 💃 😹

[MorrisJobke]

And another three requests less per page load 🚀

I was wondering why this is the case, but those files are only loaded when debug is set to true. Otherwise those files are already bundled 😉

[ChristophWurst]

I was wondering why this is the case, but those files are only loaded when debug is set to true. Otherwise those files are already bundled wink

Psssst 😉

[ChristophWurst]

But they relied on server-side bundling, hence always required the server to serve them when requested. With the webpack they are static and can be served by the webserver.

[rullzer]

But they relied on server-side bundling, hence always required the server to serve them when requested. With the webpack they are static and can be served by the webserver.

Exactly. And this is something that should not be underestimated. The bundle here can be servered statically by any webserver. Proper caching gzip etc. While the other always needed to go to the storage when serving or at least setup the whole nextcloud php env to serve the file. And it is worse when a relatively slow stroage is used (objectstore requies http requests for example).

Long story short 🚀

[MorrisJobke]

Sure - I didn't meant to made this worse than it is. Even if it just would mean that we properly organize our JS code it would be perfectly fine 👍