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

Allow multiple form data tabs on parent view #315

Merged

Conversation

luca-rath
Copy link
Contributor

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Fixed tickets -
Related issues/PRs -
License MIT

What's in this PR?

Allow multiple form data tabs on parent view

Copy link
Member

@alexander-schranz alexander-schranz left a comment

Choose a reason for hiding this comment

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

Copy link
Member

@alexander-schranz alexander-schranz left a comment

Choose a reason for hiding this comment

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

After rethinking I think we should use the key instead for action and view name as that make sure that is unique by the yaml structure. Sorry for the circumstances!

I added some suggestion so we can still avoid bc break in url and view name.

@@ -132,6 +132,7 @@ sulu_form:
dynamic_lists:
sulu_page.page_edit_form: # view key e.g. "sulu_page.page_edit_form" for sulu pages or "sulu_article.edit_form" for sulu articles.
form: # unique key mostly the same as the template key or a combination between template and property key.
Copy link
Member

Choose a reason for hiding this comment

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

as reading this command I think we should just use the key as postfix for the viewbuilder, so uniqueness is defined by the yaml key.

Admin/DynamicListAdmin.php Outdated Show resolved Hide resolved
Admin/DynamicListAdmin.php Outdated Show resolved Hide resolved
Admin/DynamicListAdmin.php Show resolved Hide resolved
Admin/DynamicListAdmin.php Outdated Show resolved Hide resolved
Resources/doc/dynamic.md Outdated Show resolved Hide resolved
@alexander-schranz alexander-schranz merged commit c3908bd into sulu:2.x Feb 2, 2022
@luca-rath luca-rath deleted the enhancement/dynamic-list-admin branch February 2, 2022 17:48
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.

2 participants