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

!!! TASK: Move ContentRepositoryId to SharedModel namespace #4891

Merged

Conversation

mhsdesign
Copy link
Member

@mhsdesign mhsdesign commented Feb 14, 2024

Originally the content repository didnt know its id but with the change #4310 we added it.
Thats why i presume the ContentRepositoryId lived in the factory namespace.
But since we now like to expose it in the ContentRepository and in the to be conceptualised NodeIdentity the idea about its location changed. The dto does no longer belong to the Factory namespace.

ContentRepositoryId should not be in that namespace o.O

Originally posted in #4868 (comment)

Upgrade instructions

Instead of referencing Neos\ContentRepository\Core\Factory\ContentRepositoryId you should reference Neos\ContentRepository\Core\SharedModel\ContentRepository

(That can as well be a simple string replace)

Review instructions

Checklist

  • Code follows the PSR-2 coding style
  • Tests have been created, run and adjusted as needed
  • The PR is created against the lowest maintained branch
  • Reviewer - PR Title is brief but complete and starts with FEATURE|TASK|BUGFIX
  • Reviewer - The first section explains the change briefly for change-logs
  • Reviewer - Breaking Changes are marked with !!! and have upgrade-instructions

@github-actions github-actions bot added the 9.0 label Feb 14, 2024
@mhsdesign mhsdesign mentioned this pull request Feb 14, 2024
35 tasks
@mhsdesign
Copy link
Member Author

Ui adjustments neos/neos-ui#3717

@mhsdesign mhsdesign marked this pull request as draft February 18, 2024 10:56
@mhsdesign mhsdesign force-pushed the task/moveContentRepositoryId-to-correct-namespace branch from e572de6 to 0b501ee Compare March 15, 2024 08:28
@mhsdesign mhsdesign marked this pull request as ready for review March 15, 2024 08:28
@mhsdesign mhsdesign force-pushed the task/moveContentRepositoryId-to-correct-namespace branch from 0b501ee to 20d28de Compare March 16, 2024 16:20
Copy link
Member

@bwaidelich bwaidelich left a comment

Choose a reason for hiding this comment

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

Makes sense, thanks for taking care!

The dto does not belong to the `Factory` namespace.
@mhsdesign mhsdesign force-pushed the task/moveContentRepositoryId-to-correct-namespace branch from 20d28de to 7c91277 Compare March 16, 2024 16:34
@kitsunet kitsunet enabled auto-merge March 16, 2024 16:35
Copy link
Contributor

@dlubitz dlubitz left a comment

Choose a reason for hiding this comment

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

OMG ... this means we also have to adopt all external packages. 🙈

@kitsunet kitsunet merged commit 4e6c2f0 into neos:9.0 Mar 16, 2024
9 checks passed
@mhsdesign
Copy link
Member Author

mhsdesign commented Mar 16, 2024

jup, but we already have to because of the workspaces and subtree tags 😅

and its not that widely used, for example pure fusion packages wont be affected.

Anke and me discussed that after this week we should probably check on a few packages ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants