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

feat(edit-ema): Show notification for duplicate contentlet in container #27335 #27351

Merged

Conversation

zJaaal
Copy link
Contributor

@zJaaal zJaaal commented Jan 17, 2024

Proposed Changes

  • Refactor insert utils functions to return whether or not the content was add
  • Add message to inform the user that the contentlet was not add because is dupe

Screenshot

Screen.Recording.2024-01-17.at.10.40.45.AM.mov

@fmontes fmontes changed the title Fix #27335: Edit EMA add notification for duplicate contentlet addition feat(edit-ema): Show notification for duplicate contentlet in container #27335 Jan 17, 2024
@fmontes fmontes enabled auto-merge January 17, 2024 19:33
@fmontes fmontes added this pull request to the merge queue Jan 17, 2024
@zJaaal zJaaal removed this pull request from the merge queue due to a manual request Jan 17, 2024
@zJaaal zJaaal added this pull request to the merge queue Jan 17, 2024
Merged via the queue into master with commit 507e128 Jan 17, 2024
15 checks passed
@zJaaal zJaaal deleted the 27335-edit-ema-add-notification-for-duplicate-contentlet-addition branch January 17, 2024 22:30
@fmontes fmontes mentioned this pull request Jan 19, 2024
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 EMA: Add Notification for Duplicate Contentlet Addition
3 participants