Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calendar invitation reply result not visually updated from attendee #1030

Closed
Arsnael opened this issue May 8, 2024 · 10 comments
Closed

Calendar invitation reply result not visually updated from attendee #1030

Arsnael opened this issue May 8, 2024 · 10 comments
Assignees

Comments

@Arsnael
Copy link
Member

Arsnael commented May 8, 2024

Related method call

Calendar/reply

Environment

Production - tmail.linagora.com

Reproducible steps

  • Person A (any email provider) sends an event invitation to person B (linagora mail)
  • Person B clicks any of the reply (yes/no/maybe) button from the email invitation
  • Person B confirms the success of person A's reply by checking their email
  • Both A & B check their calendar
    • B will see A's attend status
    • A will not see his/her own attend status

Evidents

  • Reply video
  • Inviter success confirmation & calendar visualization:

Image
Image
Image

  • Invited (attendee) calendar visualization

Image
Image

@vttranlina
Copy link
Member

can you update who is A, who is B in the Video, and the picture?
Sorry, A lot of pictures, but still confused about what you explain

@vttranlina
Copy link
Member

Summary: A invites B, but A can not see A status on A calendar?

@Arsnael
Copy link
Member Author

Arsnael commented May 8, 2024

@vttranlina from what I understood from discussing it with dat a bit was:

Alice invite Bob 
Bob reply by Tmail blue Bar
Alice can see Bob reply in all cases (Alice is openpaas or non openpaas user)
But Bob can not see his status

After checking a bit I think, if you take the current behavior on prod:

  • alice creates an envent and invites bob
  • bob receives an email asking him his attendance to this event
  • bob clicks on yes button in the blue bar
  • opens a new tab with uri params telling openpaas calendar that bob attends this event, so openpaas cal does the update at that moment for bob (I think that's the current hack in place)
  • sends a reply message with ics file on bob status to alice
  • tmail process the mail for alice, detects the ics, extract the info, send it to rabbitmq, sabre consumes it and sync for alice about bob status

I think what mobile team does for testing is that when bob clicks on yes button, they use the jmap metthod for attendance, which replies an email with the ics that then tmail treats for alice and send to sabre to update bob attendance on alice account , but not opening a new tab with calendar uri with query param and not updating bob attendance status for bob on openpaas cal

Am I correct guys?

@vttranlina
Copy link
Member

I got it, thank you.
I remember the purpose of the Jmap calendar reply was just to show the blue bar on Alice's side.
We didn't test the comunicate between tmail and openpass

@quantranhong1999
Copy link
Member

opens a new tab with uri params telling openpaas calendar that bob attends this event, so openpaas cal does the update at that moment for bob (I think that's the current hack in place)

I think it is not 100% related to the Calendar reply extension.

For example, I tested on prod (today is still using the hacky HTML extraction for the blue bar IMO):

  • quan.tranhong1999@gmail.com (Google Calendar) invites hqtran@linagora.com (OpenPaas Calendar).
  • hqtran@linagora.com click Yes on the blue bar to accept the invitation.
  • quan.tranhong1999@gmail.com (Google Calendar) updates the status of hqtran@linagora.com to Yes (attend).
  • But hqtran@linagora.com (OpenPaas Calendar) does not update his status to Yes.

Can say this issue has been there the whole time and is not related to the Calendar reply extension IMO.

@chibenwa
Copy link
Member

chibenwa commented May 8, 2024

(Disclaimer: on holiday, I read the bug description however I did not read your comments)

That's expected!

We need to modify the ICS pipeline to also transmit the ICS generated by Bob to OpenPaaS. Sending the ICS as if it was received by Bob is likely enough.

TLDR: we need a simple mailetcontainer modification for this.

If needed I will do it on monday.

@Arsnael
Copy link
Member Author

Arsnael commented May 9, 2024

I did think about it yesterday and tried to take a look on mailetcontainer.xml on prod in the cal section, but I wasn't completely sure. Might take an other look, if not support welcome monday

@Arsnael
Copy link
Member Author

Arsnael commented May 9, 2024

We need to modify the ICS pipeline to also transmit the ICS generated by Bob to OpenPaaS. Sending the ICS as if it was received by Bob is likely enough.

Would be interested to know how... The only mail that gets processed is the one received by Alice from Bob. I really don't see how you send your ICS pretending it was received by Bob too. Not really keen on testing something I aint sure about on prod as well (no calendar on staging)

@chibenwa
Copy link
Member

#1033

Please team take care of the PR from now on now that dark magic is done...

proof.mp4

@vttranlina vttranlina self-assigned this May 14, 2024
@vttranlina
Copy link
Member

I continue it on the existing Benoit PR #1033

@Arsnael Arsnael closed this as completed May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants