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

[Part-IV] TF-2812 Calendar event accept Integrate UI #2815

Merged

Conversation

tddang-linagora
Copy link
Contributor

@tddang-linagora tddang-linagora commented Apr 19, 2024

Issue

TODO

  • Test when backend is ready
  • Add demo after testing

Testing setup in preprod

  • Take an ics from prod's email, change organizer to an email from preprod, let's call email A, in the form of abc@upn.integration-open-paas.org, also update an attendee to the testing email, let's call email B, with the same form as the organizer
  • Sign in to email B, compose an email, send to self, with the attachment is the ics above
  • Due to faking of calendar event, the email content will not be qualify for the 3 buttons yes, maybe and no to appear. A fake parse has to be created. Head to lib/features/email/data/local/html_analyzer.dart, replace all code inside getListEventAction to:
return [
 EventAction(EventActionType.yes, ''),
 EventAction(EventActionType.maybe, ''),
 EventAction(EventActionType.no, ''),
];
  • Make the loading duration longer by heading to lib/features/email/domain/usecases/calendar_event_accept_interactor.dart, add await Future.delayed(const Duration(seconds: 2)); to line 19

Demo

Screen.Recording.2024-04-19.at.15.13.29.mov

@tddang-linagora tddang-linagora self-assigned this Apr 19, 2024
@tddang-linagora tddang-linagora changed the title Feature/tf 2812 calendar event accept UI TF-2812 Calendar event accept Integrate UI Apr 19, 2024
Copy link

This PR has been deployed to https://linagora.github.io/tmail-flutter/2815.

@tddang-linagora tddang-linagora changed the title TF-2812 Calendar event accept Integrate UI [Part-IV] TF-2812 Calendar event accept Integrate UI May 2, 2024
@tddang-linagora tddang-linagora force-pushed the feature/TF-2812-calendar-event-accept-ui branch from 277fef9 to 907c6b6 Compare May 3, 2024 10:38
Copy link
Member

@dab246 dab246 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Squash & Rebase commit

@hoangdat hoangdat merged commit d564b1f into feature_calendar_actions May 6, 2024
14 checks passed
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

Successfully merging this pull request may close these issues.

3 participants