-
Notifications
You must be signed in to change notification settings - Fork 324
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
Enhances spo theme list
#5346
Enhances spo theme list
#5346
Conversation
Thank you, we'll try to review it ASAP! |
Hi, @MathijsVerbeeck thank you for contributing to the next major release. Since this is a breaking change, could you please extend our v7 upgrade guidance with this breaking change? Much appreciated! 🙏 |
fc2864d
to
0111b6c
Compare
934602e
to
b7ec1d3
Compare
0111b6c
to
5d9ef64
Compare
@MathijsVerbeeck seems like something went wrong while resolving the merge conflicts. The only change in this PR is to update the upgrade guidance. Could you have a look at it please? |
I love rebasing 😄 |
@MathijsVerbeeck unfortunately this one as well will need a rebase with the latest v7 before we proceed 🙏. Thanks 👍 |
7eac0c4
to
f0e5a00
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there! Let's make a few tiny adjustments before we merge it.
167c1c5
to
151f0d4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing to add!
Merged manually, thank you! 👏 |
Closes #5231