Replace included vendor files with composer dependencies

niccofyren commented 4 years ago

WIP - Don't merge

Replaces vendors/ folder contents with composer dependencies. This includes a few version and library changes.

Known issues:

[ ] Wiki diff's are not as pretty anymore 😢 - Feel free to suggest good alternative inline diff libraries
[ ] Asset compression and cache appears to not be in use (https://github.com/gathering/wannabe/blob/master/app/Config/core.example.php#L222 and url tests below). Should we keep the assets.php anyway, or just remove? Maybe it's used in other cases than the one I tested?

To check if compression is activated; The following url should respond with minified file: https://wannabe.gathering.org/js/bootstrap.js / http://localhost:4000/js/bootstrap.js

Checklist for getting in prod:

[ ] Run composer install in prod(-like) env with new dependencies and potentially adjust (or ignore) lockfile from dev env.

niccofyren commented 4 years ago

We could merge this if none of the know issues are deal breakers. Might be worth testing locally to properly evaluate how bad the diffing is. And checking prod env to see if any of the assets options are enabled there or not.

KristianLyng commented 4 years ago

LGTM :+1: I'd prefer one more ACK, but lacking that, I'll merge it...

gathering / wannabe

Replace included vendor files with composer dependencies #23