-
Notifications
You must be signed in to change notification settings - Fork 9
Warn or straight up refuse to migrate spaces that would overwrite well known spaces #320 #385
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
Conversation
...-pro-default/src/main/java/com/xwiki/confluencepro/internal/ConfluenceFilteringListener.java
Outdated
Show resolved
Hide resolved
...-confluence-migrator-pro-ui/src/main/resources/ConfluenceMigratorPro/Code/MigrationSheet.xml
Outdated
Show resolved
Hide resolved
...nce-migrator-pro-api/src/main/java/com/xwiki/confluencepro/ConfluenceMigrationJobStatus.java
Outdated
Show resolved
Hide resolved
...-pro-default/src/main/java/com/xwiki/confluencepro/internal/ConfluenceFilteringListener.java
Outdated
Show resolved
Hide resolved
...-confluence-migrator-pro-ui/src/main/resources/ConfluenceMigratorPro/Code/MigrationSheet.xml
Outdated
Show resolved
Hide resolved
pom.xml
Outdated
| <listeneradmin.version>1.0.2</listeneradmin.version> | ||
| <xwiki.extension.category>application</xwiki.extension.category> | ||
| <confluence.version>9.88.4</confluence.version> | ||
| <confluence.version>9.88.105-SNAPSHOT</confluence.version> |
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.
This should not be merged.
On the other hand, if it gets merged, it's not the end of the world because I'm soon releasing Confluence Migrator Pro with an updated confluence.version, so this has no chance of ending up in a release.
pom.xml
Outdated
| <groupId>com.xwiki.confluencepro</groupId> | ||
| <artifactId>application-confluence-migrator-pro</artifactId> | ||
| <version>1.36.0-rc-2-SNAPSHOT</version> | ||
| <version>9.88.4</version> |
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.
@rbancescu this needs to be reverted
Because this changes depends on other changes related to confluence-xml, included in xwiki-contrib/confluence#78 , once the PR is merged and a release is made, the dependency on org.xwiki.contrib.confluence:confluence-xml should be updated to include the new code before this PR is merged!