Skip to content

Procedure to update from 4.6 to 5.0 #2760

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

Open
wants to merge 50 commits into
base: 5.0
Choose a base branch
from
Open

Procedure to update from 4.6 to 5.0 #2760

wants to merge 50 commits into from

Conversation

adriendupuis
Copy link
Contributor

@adriendupuis adriendupuis commented May 27, 2025

Question Answer
JIRA Ticket
Versions 5.0, 4.6, 3.3
Edition

Procedure to update from 4.6 to 5.0.

Previews:

CI implementation:

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Code samples are working
  • PHP code samples have been fixed with PHP CS fixer
  • Added link to this PR in relevant JIRA ticket or code PR

Copy link

@adriendupuis adriendupuis marked this pull request as ready for review July 16, 2025 13:30
Copy link
Contributor

@mnocon mnocon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for preparing this, it's already great 🤩

A couple of suggesitons from me, some of them we can test in the future if needed.

category: Custom
- thumbnail: /bundles/ibexaicons/img/all-icons.svg#about
+ thumbnail: /bundles/ibexaicons/img/all-icons.svg#info-square
```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ending feels abrupt, I'd end here on a positive note 😄

Maybe something like:

Suggested change
```
```
Finish the update process:
```
composer run post-install-cmd
```
Your project is now running the latest version of Ibexa DXP.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mnocon

I reworked the end in 9f4124a...a3e1ffa

It's true that composer run-script post-update-cmd was run only once in the middle.

@adriendupuis adriendupuis requested a review from mnocon July 21, 2025 13:31
@adriendupuis adriendupuis requested a review from Steveb-p July 21, 2025 15:05
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.

3 participants