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

3598 - Custom page for entity view #3617

Merged
merged 118 commits into from
Jun 1, 2021
Merged

Conversation

Zasa-san
Copy link
Collaborator

@Zasa-san Zasa-san commented Apr 20, 2021

fixes #3598

PR checklist:

  • Update READ.me ?
  • Update API documentation ?

QA checklist:

  • Smoke test the functionality described in the issue
  • Test for side effects
  • UI responsiveness
  • Cross browser testing
  • Code review

For MVP we handle editing/deleting pages used for entity view the following way:
If a user wants to stop using an existing page as entity view, and they try to toggle off the functionality on the page editor or delete the page, they will get an error message explaining that the page is in use with a list of the templates using them. This way they can remove the page from the templates using it, then proceed to toggle entity view off or delete it.

@Zasa-san Zasa-san force-pushed the 3598-custom-entity-view branch from 6938879 to 1ad6b5d Compare May 31, 2021 18:45
@Zasa-san Zasa-san marked this pull request as ready for review May 31, 2021 18:55
Zasa-san added a commit that referenced this pull request May 31, 2021
@daneryl daneryl merged commit dc48b51 into development Jun 1, 2021
@daneryl daneryl deleted the 3598-custom-entity-view branch June 1, 2021 09:49
Zasa-san added a commit that referenced this pull request Jun 1, 2021
Zasa-san added a commit that referenced this pull request Jun 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom page for entity view (20pts)
3 participants