474275 - Alarms don't store on storage calendar

Status: RESOLVED FIXED

(Calendar :: Provider: Local Storage, defect)

Description

[Andreas Treumann]

STEPS TO REPRODUCE:
===================

- create an recurring event with alarm
- wait...

RESULT:
=======

- no alarm icon in calendar view
- no alarm comes up

EXPECTED RESULT:
================

- the user should get an alarm

REPRODUCIBLE:
=============

- always


Defect in  lightning build 20090118, works in build 20090116.

Comment 1

[Philipp Kewisch [:Fallen][📱📆]]

Attachment: Fix - v1

Since the fix for this is so obvious and small, I'm r+'ing this myself.

Comment 2

[Philipp Kewisch [:Fallen][📱📆]]

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/636817f502d0>

-> FIXED

Comment 3

[Martin Schröder [:mschroeder]]

Push to comm-central has not been successful! -> REOPENED

Comment 4

[Philipp Kewisch [:Fallen][📱📆]]

Although I see the hg link doesn't work, hg log shows that the patch is there and after an update the added line is also there. Did I miss anything?

Comment 5

[Philipp Kewisch [:Fallen][📱📆]]

Seems I did qpush instead of push when committing. 

Pushed to comm-central <http://hg.mozilla.org/comm-central/rev/3858b26cdd91>

-> FIXED

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Seen the following problem:

Had some alarm reminder for recurring events in the reminder pop-up open and don't dismissed them, closed Calendar, upgrade run and introduced missing alarm bug > no reminders

upgrade to latest tbox-build > none of the events in the reminder pop-up, no alarm icon in the calendar pane, event edit dialog still shows event as recurring with reminder.

Interesting: The reminder for a not-recurring event showed up again as expected.

Comment 7

[Philipp Kewisch [:Fallen][📱📆]]

These bugs are likely targeted at Lightning 1.0b1, not Lightning 1.0. If this change was done in error, please adjust the target milestone to its correct value. To filter on this bugspam, you can use "lightning-10-target-move".