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

Fix #26314: Edit Page UrlContentMap Fix regeneration of menu items for new dropdown #26453

Merged
merged 4 commits into from Oct 20, 2023

Conversation

zJaaal
Copy link
Contributor

@zJaaal zJaaal commented Oct 18, 2023

🤖 Generated by Copilot at 6f36e95

Summary

✅✨🔧

This pull request adds conditional logic and tests for displaying the menu items for editing the URL content map in the edit page mode. This is part of the feature to allow users to edit the content associated with a URL in both the content and the SEO tabs. The changes affect the dot-edit-page-state-controller.component.ts, dot-edit-page-state-controller-seo.component.ts, and their respective spec files.

Sing, O Muse, of the skillful coders who devised
A wondrous feature for the web, to edit and map
The content of each URL, with menus and flags
That change with the page state, as swift as Hermes' wings.

Walkthrough

  • Add menu items for editing and removing URL content map in edit page mode and SEO tab (link, link, link, link)
  • Initialize menuItems array to empty array in both components to avoid errors (link, link)

Screenshots

Screen.Recording.2023-10-18.at.1.43.19.PM.mov

@dotcms-sonarqube
Copy link

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@fmontes fmontes merged commit f1c2d90 into master Oct 20, 2023
24 checks passed
@fmontes fmontes deleted the 26314-edit-options-not-showing-on-urlcontent-page branch October 20, 2023 17:08
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.

Edit Options not Showing on UrlContent Page
5 participants