linagora / tmail-backend

GNU Affero General Public License v3.0
31 stars 17 forks source link

Reply ICS should generate new DTSTAMP / LAST-MODIFIED #975

Closed quantranhong1999 closed 3 months ago

quantranhong1999 commented 4 months ago

Why

GIVEN Alice (Google Calendar) invites Bob (TMail user) to join the event X. Bob replies an mail with an ICS file with PARTSTAT=ACCEPTED, then Bob's accept status would be updated in Google Calendar of Alice.

WHEN Bob updates the invitation reply e.g. an ICS file with PARTSTAT=DECLINED THEN today Bob's declined status would not be updated in Google Calendar of Alice.

Reason: today likely the first ICS reply and the second ICS reply have the same DTSTAMP / LAST-MODIFIED. Therefore Google Calendar would not update the status upon the second ICS reply.

How

We should create new values for DTSTAMP / LAST-MODIFIED for ICS reply.

Arsnael commented 3 months ago

Good catch

quantranhong1999 commented 3 months ago

https://github.com/linagora/tmail-backend/pull/979