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

chore(edit ema): Move edit dialog to one place #27526 #27542

Merged
merged 23 commits into from
Feb 12, 2024

Conversation

zJaaal
Copy link
Contributor

@zJaaal zJaaal commented Feb 7, 2024

Proposed changes

  • Create DotEmaDialogComponent to centralize Iframe Dialogs in EMA
  • Clean edit-ema-store
  • Integrate new component where is needed
  • Also fixed an small bug that prevented the dialog to close when adding a duplicated contentlet

Screenshot

Screen.Recording.2024-02-08.at.6.01.08.PM.mov

@zJaaal zJaaal linked an issue Feb 7, 2024 that may be closed by this pull request
@zJaaal zJaaal requested a review from fmontes February 7, 2024 20:56
@zJaaal zJaaal marked this pull request as ready for review February 8, 2024 21:03
@zJaaal zJaaal requested a review from fmontes February 12, 2024 16:35
@zJaaal zJaaal enabled auto-merge February 12, 2024 17:32
@zJaaal zJaaal added this pull request to the merge queue Feb 12, 2024
Merged via the queue into master with commit b3b629d Feb 12, 2024
41 checks passed
@zJaaal zJaaal deleted the 27526-edit-ema-move-edit-dialog-to-one-place branch February 12, 2024 19:34
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: 🧹 Move Edit Dialog to One Place
3 participants