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

headless models as parents #1936

Closed
giseleblair opened this issue Mar 17, 2023 · 5 comments · Fixed by #1947
Closed

headless models as parents #1936

giseleblair opened this issue Mar 17, 2023 · 5 comments · Fixed by #1947
Assignees
Labels
bug Something isn't working

Comments

@giseleblair
Copy link
Contributor

It looks like the UI's content navigation tree now displays nested headless models properly. Please allow for headless models to be selected in the Schema app.
The feature was removed as requested here when the UI did not display nested headless sets properly in the content nav tree. This looks to have been updated and now the functionality should be brought back in the UI.

@giseleblair giseleblair added the bug Something isn't working label Mar 17, 2023
@agalin920
Copy link
Contributor

agalin920 commented Mar 20, 2023

Shouldn't this be labeled as an enhancement since this is expected behavior that needs updated?

@giseleblair
Copy link
Contributor Author

Now that the UI displays the nested items in the navigation tree, the user does expect to be able to nest any and all models. This is not an enhancement since it is part of the api's current capabilities.

@agalin920 agalin920 self-assigned this Mar 22, 2023
@agalin920
Copy link
Contributor

agalin920 commented Mar 22, 2023

I believe adding a 6- as a parentZUID to a single page model rejects the create item from that model api call. We might need to filter these based on model type.

@markelmad Can you verify that for me?

cc: @theofficialnar

@markelmad
Copy link

I believe adding a 6- as a parentZUID to a single page model rejects the create item from that model api call. We might need to filter these based on model type.

@markelmad Can you verify that for me?

cc: @theofficialnar

Didn't find any validation/restriction check for this one for both CM and item.

@agalin920
Copy link
Contributor

@markelmad Thanks for verifying

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants