-
-
Notifications
You must be signed in to change notification settings - Fork 136
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
FEATURE: Extract workspace metadata and user-assignment to Neos #3838
base: 9.0
Are you sure you want to change the base?
FEATURE: Extract workspace metadata and user-assignment to Neos #3838
Conversation
…ser-assignment-to-neos
…orkspace-metadata-and-user-assignment-to-neos
@@ -14,7 +14,7 @@ | |||
"require": { | |||
"neos/flow-development-collection": "9.0.x-dev as 9.0", | |||
|
|||
"neos/neos-development-collection": "9.0.x-dev as 9.0", | |||
"neos/neos-development-collection": "feature/4726-extract-workspace-metadata-and-user-assignment-to-neos-dev as 9.0", |
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.
revert me :)
$command['contentRepositoryId'] = $contentRepositoryId->value; | ||
$command['siteId'] = $this->nodeService->deserializeNodeAddress( | ||
$siteNodeAddress = $this->nodeService->deserializeNodeAddress( | ||
$command['siteId'], | ||
$contentRepositoryId | ||
)->nodeAggregateId->value; | ||
$command = DiscardChangesInSite::fromArray($command); |
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.
i know this currently looks weird but this was just introduced by Bernhard and Wilhelm and we plan - as the comment says @todo send from UI
- to send the array shape directly from the neos ui and then call DiscardChangesInSite::fromArray
just to have type safety. Eventually each command will have its own proper command handler etc but right now its in an in between state. So i would like to keep the usages to the commands.
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.
reintroduced the usages. Otherwise we would also have ugly conflicts with #3858
Counter-part to neos/neos-development-collection#5146
Related: neos/neos-development-collection#4726