-
Notifications
You must be signed in to change notification settings - Fork 3
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
[MWPW-157579] Event Title Duplication Prevention #200
Conversation
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #200 +/- ##
==========================================
+ Coverage 67.22% 67.36% +0.13%
==========================================
Files 2 2
Lines 238 239 +1
==========================================
+ Hits 160 161 +1
Misses 78 78 ☔ View full report in Codecov by Sentry. |
@@ -94,7 +94,7 @@ export default function init(el) { | |||
generateToolTip(r); | |||
break; | |||
case 1: | |||
await decorateTextfield(r, { id: 'info-field-event-title' }); | |||
await decorateTextfield(r, { id: 'info-field-event-title' }, 'This event title is already taken'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably we can get the message from word document? and can it be - "This event title is already taken for a given event series, date and venue combination"
looks like there is conflict... after resolving it, this can get merged |
Resolves: MWPW-157579
Test URLs: