Closed Bug 788267 Opened 9 years ago Closed 9 years ago

Replace calRecurrenceDate.cpp with a JS implementation

Categories

(Calendar :: Internal Components, defect)

Lightning 1.7
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Fallen, Assigned: Fallen)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Fix - v1 (obsolete) β€” β€” Splinter Review
No description provided.
Attachment #658213 - Flags: review?(matthew.mecca)
Comment on attachment 658213 [details] [diff] [review]
Fix - v1

This conflicts with my own patch on another bug. Let me fix that before review :)
Attachment #658213 - Attachment is obsolete: true
Attachment #658213 - Flags: review?(matthew.mecca)
Attached patch Fix - v2 β€” β€” Splinter Review
This should do better
Attachment #667416 - Flags: review?(matthew.mecca)
Comment on attachment 667416 [details] [diff] [review]
Fix - v2

Looks good. r=mmecca
Attachment #667416 - Flags: review?(matthew.mecca) → review+
Pushed to comm-central changeset eb66cf3da706
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.0
You need to log in before you can comment on or make changes to this bug.