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: Fix documentation of cr:import #4740

Merged
merged 2 commits into from
Nov 12, 2023
Merged

TASK: Fix documentation of cr:import #4740

merged 2 commits into from
Nov 12, 2023

Conversation

dlubitz
Copy link
Contributor

@dlubitz dlubitz commented Nov 12, 2023

This adjusts the documentation of `cr:import

@dlubitz dlubitz enabled auto-merge November 12, 2023 14:10
@dlubitz dlubitz added the 9.0 label Nov 12, 2023
@dlubitz dlubitz self-assigned this Nov 12, 2023
@mhsdesign mhsdesign changed the title Fix documentation TASK: Fix documentation of cr:import Nov 12, 2023
@github-actions github-actions bot added the Task label Nov 12, 2023
@mhsdesign mhsdesign disabled auto-merge November 12, 2023 14:57
@mhsdesign mhsdesign merged commit 983db91 into 9.0 Nov 12, 2023
7 checks passed
@mhsdesign mhsdesign deleted the dlubitz-patch-1 branch November 12, 2023 14:57
@@ -76,7 +76,7 @@ public function exportCommand(string $path, string $contentRepository = 'default
/**
* Import the events from the path into the specified content repository
*
* @param string $path The path for storing the result
* @param string $path The path of the stored events like resource://Neos.Demo/Private/Content
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mhsdesign But this is not correct. The path is enough like Packages/Sites/Neos.Demo/Private/Content

Copy link
Member

Choose a reason for hiding this comment

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

hmm i prefer the abstraction via the resource manager as this makes it a detail where the package is stored inside Packages

And i tested it - it works ^^

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

Successfully merging this pull request may close these issues.

2 participants