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(edit-ema): 🐛 Custom Event select-contentlet is triggering twice on ng-contentlet-selector #26958 #27363

Conversation

KevinDavilaDotCMS
Copy link
Contributor

@KevinDavilaDotCMS KevinDavilaDotCMS commented Jan 18, 2024

Proposed Changes

  • Now only trigger one time when user clicks on select button

Updated video:

Screen.Recording.2024-01-18.at.11.44.45.mov

Note: In the video i show console.logs, but is only for show the problem, console.logs are not in PR

This internal change doesnt break the old edit mode

Screen.Recording.2024-01-18.at.11.59.58.mov

Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@fmontes
Copy link
Member

fmontes commented Jan 18, 2024

YASSSS!!!!

@fmontes fmontes changed the title Fixed trigger twice on ng-contentlet-selector fix(edit-ema): 🐛 Custom Event select-contentlet is triggering twice on ng-contentlet-selector #26958 Jan 18, 2024
@fmontes fmontes added this pull request to the merge queue Jan 18, 2024
Merged via the queue into master with commit 219a96c Jan 18, 2024
15 checks passed
@fmontes fmontes deleted the 26958-edit-ema-custom-event-select-contentlet-is-triggering-twice-on-ng-contentlet-selector branch January 18, 2024 17:50
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: 🐛 Custom Event select-contentlet is triggering twice on ng-contentlet-selector
3 participants