Changes proposed in this pull request:
Please refer to the issue for a detailed plan of the changes introduced here.
Compat API has been replaced with simple forum/forum.tscommon/common.ts and admin/admin.ts that import all modules so that they are included in the final bundle regardless of i they are used internally or not.
Necessity
[x] Has the problem that is being solved here been clearly explained?
[x] If applicable, have various options for solving this problem been considered?
[x] For core PRs, does this need to be in core, or could it be in an extension?
[x] Are we willing to maintain this for years / potentially forever?
Confirmed
[ ] Frontend changes: tested on a local Flarum installation.
[ ] Backend changes: tests are green (run composer test).
[ ] Core developer confirmed locally this works as intended.
[ ] Tests have been added, or are not appropriate here.
Fixes #3833
Changes proposed in this pull request: Please refer to the issue for a detailed plan of the changes introduced here.
forum/forum.ts
common/common.ts
andadmin/admin.ts
that import all modules so that they are included in the final bundle regardless of i they are used internally or not.Necessity
Confirmed
composer test
).