dotCMS / core

Headless/Hybrid Content Management System for Enterprises
http://dotcms.com
Other
861 stars 468 forks source link

Edit Page V2: Allow User to Choose Content Editing for Individual or All Pages #27551

Closed fmontes closed 2 months ago

fmontes commented 8 months ago

Parent

27546

User Story

As a content editor, I want to be informed if the content I am editing on a page will affect other pages, so that I can make an informed decision about whether to edit the content for all pages or only for the current page.

image
### Acceptance Criteria
- [x] When the user clicks on the "Edit" button for a content item on a page, a dialog should be displayed.
- [x] Dialog only shows if the edit content is being used in another page
- [x] The functionality should be consistent with the existing "Edit" functionality in the old edit page.
- [x] The feature should be implemented in the new edit page.
- [x] This functionality should work for headless pages as well

Proposed Objective

Static content: "Core Features"

Proposed Priority

Static content: "Priority 3 - Average"

External Links... Slack Conversations, Support Tickets, Figma Designs, etc.

Static content: "N/A"

rjvelazco commented 8 months ago

QA Team

This issue is waiting for this one: https://github.com/dotCMS/core/issues/27658

josemejias11 commented 7 months ago

Failed: Tested on master_f3b48ba, Docker, macOS 13.0, FF v121.0.1

According with @rjvelazco the property needs to be restore with the use of the new sdk

rjvelazco commented 7 months ago

@josemejias11 Fix: https://github.com/dotCMS/core/pull/28030

josemejias11 commented 7 months ago

Approved: Tested on dotcms-qa-master.dotcms, Docker, macOS 13.0, FF v121.0.1

weiyiwang-dotcms commented 6 months ago

Hi, I have a support case https://dotcms.zendesk.com/agent/tickets/115969 related to this. Customer is on v 24.04.05, and is using EMA. They are not receiving the popup when they click on the edit button, even though the content is on multiple pages. screen-capture-edit-modal-popup-not-displayed.webm

josemejias11 commented 6 months ago

Hi @weiyiwang-dotcms,

Currently this functionality is working properly on https://dotcms-qa-master.dotcms.site/c which is the environment where EMA was configured for testing.

Screenshot 2024-04-16 at 2 29 21 PM

@rjvelazco can you please check on this and confirm if is a version issue or something else. Thank you so much!

rjvelazco commented 6 months ago

It might be due to the SDK being out of synchrony. We had issues publishing the SDK on that version. Do we want to port it in release-24.04.05? Or can we update them to release-24.04.16? @weiyiwang-dotcms @fmontes @josemejias11

weiyiwang-dotcms commented 6 months ago

Hi, the issue is still being reported by the customer. Here is a video that captures the issue. screen-capture-modal-popup-still-not-displayed.webm

bryanboza commented 6 months ago

I think here the problem reported by @weiyiwang-dotcms is in the old editor, this card is related to the new editor. Then we can close this one for now.

weiyiwang-dotcms commented 5 months ago

In what version of dotCMS does the new editor become available? Is the new editor available in release-24.04.16?

rjvelazco commented 3 months ago

@nollymar Should we close this card and open a new one for the old editor to check the case @weiyiwang-dotcms reported?