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

refactor: flarum 2.0 upgrade #89

Draft
wants to merge 11 commits into
base: master
Choose a base branch
from
Draft

Conversation

SychO9
Copy link
Contributor

@SychO9 SychO9 commented Nov 2, 2024

Upgrades the extension to be compatible with Flarum 2.0

  • This repository was used to test the reliability of the Flarum 2.0 upgrade tool and reinforce it. The rest of it was updated manually and contributed for your convenience.
  • I will likely be posting multiple PRs like these across other repositories, and therefore will not follow-up on any changes.
  • Feel free to make changes to this branch, or close the PR and re-do the upgrade yourself if you prefer.
  • At the very least, this can be used as an example to guide your process either here or in other extensions.

Note

I recommend making a new branch specifically for 2.x releases, and basing this PR off of it, as you will likely want to still support 1.x for a while longer.

Tip

Checkout the upgrade guide: https://docs.flarum.org/2.x/extend/update-2_0


Marked as draft, but you can essentially do as you please with this.
Thanks!

Start by formatting your code with Prettier.
Update dependencies to Flarum 2.0 compatible versions.
Update the extension infrastructure
Some Flarum modules are now lazy loaded. Extending them requires a different approach.
Miscellaneous frontend changes
Miscellaneous backend changes
Flarum 2.0 completely changes the JSON:API implementation
Flarum 2.0 uses PHPUnit 11.
Many variables have been renamed to light/dark specific names and most are now used as CSS variables instead.
@SychO9 SychO9 marked this pull request as draft November 9, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant