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

Edit EMA: Add Notification for Duplicate Contentlet Addition #27335

Closed
4 tasks done
Tracked by #26644
fmontes opened this issue Jan 16, 2024 · 0 comments · Fixed by #27351
Closed
4 tasks done
Tracked by #26644

Edit EMA: Add Notification for Duplicate Contentlet Addition #27335

fmontes opened this issue Jan 16, 2024 · 0 comments · Fixed by #27351

Comments

@fmontes
Copy link
Member

fmontes commented Jan 16, 2024

Parent

#26644

User Story

As a dotCMS user, I want to be notified when I try to add the same contentlet twice to the same container so that I am aware that it is not possible.

Tasks

Preview Give feedback

Proposed Objective

Static content: "Core Features"

Proposed Priority

Static content: "Priority 3 - Average"

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

We already have a catchError in the savePage request because it is failing silently.

Assumptions & Initiation Needs

  • The user should have appropriate permissions to add contentlets to containers.
  • The system should be able to detect duplicate contentlets in a container.

Quality Assurance Notes & Workarounds

  • Verify that the toast notification is displayed when trying to add a duplicate contentlet.
  • Confirm that the notification clearly states that the contentlet cannot be added again.
  • Ensure that the notification disappears automatically
@zJaaal zJaaal self-assigned this Jan 16, 2024
@zJaaal zJaaal mentioned this issue Jan 16, 2024
@zJaaal zJaaal moved this from New to In Progress in dotCMS - Product Planning Jan 17, 2024
@zJaaal zJaaal moved this from In Progress to In Review in dotCMS - Product Planning Jan 17, 2024
github-merge-queue bot pushed a commit that referenced this issue Jan 17, 2024
#27335 (#27351)

* add flow for dupe contentlets

* add test cases

* remove icons from confirm dialog buttons

* fix for empty containers

* add more life span for the dupe notification

* fix test

* Update core-web/libs/portlets/edit-ema/portlet/src/lib/dot-ema-shell/store/dot-ema.store.ts

* add url sanitizer and test cases

---------

Co-authored-by: Freddy Montes <[email protected]>
@github-project-automation github-project-automation bot moved this from In Review to Done in dotCMS - Product Planning Jan 17, 2024
@fmontes fmontes moved this from Done to Internal QA in dotCMS - Product Planning Jan 18, 2024
@fmontes fmontes moved this from Internal QA to QA - Backlog in dotCMS - Product Planning Jan 19, 2024
@fmontes fmontes moved this from QA - Backlog to Done in dotCMS - Product Planning Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants