876796 - Editing a Google Calendar event on the server to have a different hours alarm amount does not change the client on sync

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Calendar, defect)

Description

[Jason Smith [:jsmith]]

Build: B2G 18 5/28
Device: Unagi

STR

1. Import a google calendar with some events and alarms
2. On the Google Calendar site, change one of the events to have a different hours amount for the alarm on a non-all day event
** Example - 3 hours --> 2 hours
3. Sync the calendar on the Gaia Calendar app
4. Check the event that changed in the Gaia Calendar app

Expected

The event should have the updated alarm time amount in hours.

Actual

The event retains the old alarm time in hours.

Comment 1

[James Lal [:lightsofapollo]]

This is the same problem where google does not update their E-Tags so we cannot detect the change... We can investigate more for a possible work around.

Comment 2

[James Lal [:lightsofapollo]]

I am fairly sure this is a async/race issue... we immediately show the html content that contains the event details but update it once we are able to fetch the event details from the cache/database (which is done async).

We can "fix" this by hiding? event details until they are updated.

Comment 3

[James Lal [:lightsofapollo]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/10229

Pointer to Github pull-request

Comment 4

[Angela Hubenya]

I saw this happening a multiple of times when the reminder in Google calendar is changed to a time that is not listed in the drop down of the Calendar app under the "remind me" option on the device. For example: 2 days, 30 hours, 25 min

Unagi Build ID: 20130618070211
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/a199b1109860
Gaia: 3e9090894daaa1c7f894a1dcc1026b21f889eadc
Platform Version: 18.0

Comment 5

[James Lal [:lightsofapollo]]

in master https://github.com/mozilla-b2g/gaia/commit/f9561dbe1369a8b705babfa9bac2d1cc8572e606

Comment 6

[James Lal [:lightsofapollo]]

v1-train: a597f64fa7272ed18375e9fcf73176d6e15f179cv1
v1.1.0hd: a687d927fe372a237c7b3cdb1bb44dfcb69d8000

Comment 7

[Jason Smith [:jsmith]]

Bug isn't fixed. Filed bug 887871 as a followup.

Comment 8

[Gareth Aye [:gaye] (back from PTO)]

v1.1.0hd: a687d927fe372a237c7b3cdb1bb44dfcb69d8000

Comment 9

[John Ford [:jhford] CET/CEST Berlin Time]

v1.1.0hd: a687d927fe372a237c7b3cdb1bb44dfcb69d8000