Skip to content

Conversation

alexander-schranz
Copy link
Member

@alexander-schranz alexander-schranz commented Apr 14, 2025

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets fixes #
Related issues/PRs #
License MIT
Documentation PR sulu/sulu-docs#

What's in this PR?

Add hint about jackrabbit.

Why?

Make it clearer what for apache jackrabbit based projects the migration configuration is required and that jackrabbit is after the upgrade not longer needed. /cc @datenwerk-at

@alexander-schranz alexander-schranz force-pushed the enhancement/upgrade-3-0-jackrabbit branch from 2d5fbd9 to 1fbc6a2 Compare April 14, 2025 15:45
Co-authored-by: mamazu <14860264+mamazu@users.noreply.github.com>
@Prokyonn Prokyonn enabled auto-merge (squash) April 14, 2025 23:24
@Prokyonn Prokyonn merged commit 39c0d20 into sulu:3.0 Apr 14, 2025
9 checks passed
@mamazu
Copy link
Contributor

mamazu commented Apr 15, 2025

Would this note about how to configure the migration bundle also make sense in the migration bundle docs as well. I was trying out Sulu3 yesterday and was confused how this bundle should be set up (because there's no docs over there).

@alexander-schranz alexander-schranz deleted the enhancement/upgrade-3-0-jackrabbit branch April 15, 2025 09:31
@Prokyonn
Copy link
Member

Would this note about how to configure the migration bundle also make sense in the migration bundle docs as well. I was trying out Sulu3 yesterday and was confused how this bundle should be set up (because there's no docs over there).

@mamazu yes the documentation of the migration bundle should definitely be updated, I didn't have time yet to do it. But the update path with the configuration and the command should be part of the documentation.

@alexander-schranz
Copy link
Member Author

I think it make sense to copy the UPGRADE.md part into the README of the bundle.

mamazu added a commit to mamazu/SuluPHPCRMigrationBundle that referenced this pull request Apr 15, 2025
Copying over the stuff from sulu/sulu#7892
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