Can't create a Custom recurring Monthly event

RESOLVED FIXED in 5.3

Status

Calendar
Dialogs
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: WaltS48, Assigned: Decathlon)

Tracking

({regression})

Lightning 5.3
regression

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:50.0) Gecko/20100101 Firefox/50.0
Build ID: 20160920155715

Steps to reproduce:

Created an event for the first Tuesday of the month (Oct)
Set the Start and End times
Selected Custom... for Repeat
Selected Monthly for the Recurrence pattern
Selected The First Tuesday
Selected Create 5 Appointments for Range of recurrence
Clicked OK



Actual results:

The Preview showed the event occurring on the 1st Tuesday in October and the 4th of every following month, not the 1st Tuesday of the following months
Nothing happened when I clicked OK


Expected results:

The Preview should have showed the event recurring every 1st Tuesday of the following months
Clicking OK should have closed the Edit Recurrence dialog box allowing me to Save and Close the Event.
(Assignee)

Comment 1

2 years ago
Created attachment 8794099 [details] [diff] [review]
patch - v1

Confirmed.
A message "TypeError: aElement is null" appears in the console.

There is a typo in the file calendar-event-dialog-recurrence.js.
Regression from bug 1280898.
Assignee: nobody → bv1578
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #8794099 - Flags: review?(makemyday)
Attachment #8794099 - Flags: approval-calendar-aurora?(philipp)
(Assignee)

Updated

2 years ago
Blocks: 1280898
Keywords: regression
OS: Unspecified → All
Hardware: Unspecified → All

Comment 2

2 years ago
Comment on attachment 8794099 [details] [diff] [review]
patch - v1

Review of attachment 8794099 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks for the quick patch, r=me
Attachment #8794099 - Flags: review?(makemyday) → review+
Comment on attachment 8794099 [details] [diff] [review]
patch - v1

Blanket a=me for eslint regressions as they all need to go to aurora anyway.
Attachment #8794099 - Flags: approval-calendar-aurora?(philipp) → approval-calendar-aurora+
Keywords: checkin-needed

Comment 4

2 years ago
https://hg.mozilla.org/comm-central/rev/af97c06e4a9465b2292a7f69c8006ba8261fbc05
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 5.4
Version: Lightning 5.4 → Lightning 5.3
Target Milestone: 5.4 → 5.3
You need to log in before you can comment on or make changes to this bug.