[Calendar] Zero based alarms not displaying on event view

VERIFIED FIXED

Status

Firefox OS
Gaia::Calendar
P1
normal
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: kgrandon, Assigned: kgrandon)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Bug Flags:
in-moztrap +

Firefox Tracking Flags

(blocking-b2g:leo+, b2g18 fixed)

Details

(Whiteboard: [LOE:S])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Cloning this bug so flags will stick as this needs to go with the event bugs.

I made a mistake somewhere and alarms that start at the beginning of the event are not displaying on the view event page.
(Assignee)

Comment 1

5 years ago
Created attachment 724118 [details]
Github pull request pointer
Attachment #724118 - Flags: review?(jlal)
No longer blocks: 838032
(Assignee)

Comment 2

5 years ago
Landed in master: https://github.com/mozilla-b2g/gaia/commit/f65f5c7182c62240af5d7c41e904db9df1f11135
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Keywords: verifyme
I was not able to uplift this bug to v1-train.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1-train, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1-train
  git cherry-pick -x -m1 f65f5c7182c62240af5d7c41e904db9df1f11135
  <RESOLVE MERGE CONFLICTS>
  git commit
Flags: needinfo?(kgrandon)
Attachment #724118 - Flags: review?(jlal) → review+
(Assignee)

Comment 4

5 years ago
Uplifted to v1-train: 25a6f27f5270156a217c7d0f3234946cf62a5354
Flags: needinfo?(kgrandon)
status-b2g18: --- → fixed
Completed a lot of exploratory testing on this today and filed some bugs appropriately. There's more formal testing that will come from this soon, but we can mark this verified for now that this definitely has been initially looked at.
Status: RESOLVED → VERIFIED
Keywords: verifyme

Updated

5 years ago
Flags: in-moztrap?

Updated

5 years ago
Flags: in-moztrap? → in-moztrap?(cschmoeckel)

Comment 6

5 years ago
In MozTrap added Calendar Suite Test Case #9011 - [Calendar] Verify that alarms that start at the time of the event are displayed on the View Event page
Flags: in-moztrap?(cschmoeckel) → in-moztrap+
You need to log in before you can comment on or make changes to this bug.