-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Display scheduled call popup #60986
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
Display scheduled call popup #60986
Conversation
Hey! I see that you made changes to our Form component. Make sure to update the docs in FORMS.md accordingly. Cheers! |
@allroundexperts Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
Any idea on how to fix this test issue? |
@MonilBhavsar Should we assign @jjcoffee here for tests given that they reviewed the first PR? |
Yes, sounds good to me. They also have access to the doc and probably reviewed it |
Are there any screenshots or videos ready for design review? |
@shawnborton I will let you know when they are ready. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A note to test this PR for all possible timezones as there is a discrepancy in timezone names returned by Calendly versus the library we use at server to convert UTC time to local time cc @parasharrajat @jjcoffee
Let's 🚢 it |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by https://github.com/MonilBhavsar in version: 9.1.48-0 🚀
|
🚀 Deployed to staging by https://github.com/MonilBhavsar in version: 9.1.48-0 🚀
|
Hi @parasharrajat I think we can't complete this PR as we don't have accounts with a guide assigned. I tried with Expensifail and gmail account. Let me know if I something missed.
|
@MonilBhavsar can you please help here? |
@izarutskaya if you can share the email, I can assign the QA guide. |
@MonilBhavsar Thank you, I added |
Nice! 👍 |
🚀 Deployed to staging by https://github.com/MonilBhavsar in version: 9.1.49-5 🚀
|
@MonilBhavsar We can't verify Step 14 onwards because there is no I created account for team [email protected]. Can you please assign the QA guide? We will try check PR this way bandicam.2025-05-22.18-40-54-942.mp4 |
[email protected] is assigned to QA guide |
Looks like the account in video is in |
Yes, all Expensifail account is in beta, as I remember. I don't see guide in [email protected] Recording.3080.mp4maybe need to try with gmail? |
Yes, a gmail account should work. I can help to assign QA guide to it |
@MonilBhavsar Created gmail account [email protected] |
And this one please [email protected] |
@izarutskaya done |
@MonilBhavsar Seems we can't check this PR. Unfortunately we don't see guide in our accounts Recording.3081.mp4 |
If we add Recording.3082.mp4 |
I can check |
@izarutskaya Can you please make that workspace which has guide as default and try again.? |
Okay, I was able to QA this with a bit of workaround. The schedule call button is being displayed only for the active workspaces, and since we can't set workspace as active in newDot, we are not being see it always. Screen.Recording.2025-05-22.at.9.17.04.PM.mov |
🚀 Deployed to production by https://github.com/arosiclair in version: 9.1.50-0 🚀
|
Explanation of Change
Design doc
https://docs.google.com/document/d/1Q_FsNMTX9vKx_pWD75011atzJNV6kGUCBItqyHWVvaw/edit?tab=t.0#bookmark=kix.gbv1cxd24t1i
Fixed Issues
$ #59748
PROPOSAL: https://docs.google.com/document/d/1Q_FsNMTX9vKx_pWD75011atzJNV6kGUCBItqyHWVvaw/edit?tab=t.0#bookmark=kix.gbv1cxd24t1i
Tests
Prerequisites
Steps
Schedule Call
button.date&time
is shown indate&time
row.schedule call
button is changed tocall scheduled
.Reschedule
, verify that it takes you to the calendly page.cancel
, verify that it takes you to the calendly cancel page.Schedule Call
is visible again.Offline tests
User can not take this action while offline so we will show FullScreenOfflineLoader.
QA Steps
Prerequisites
Steps
Schedule Call
button.date&time
is shown indate&time
row.schedule call
button is changed tocall scheduled
.Reschedule
, verify that it takes you to the calendly page.cancel
, verify that it takes you to the calendly cancel page.Schedule Call
is visible again.PR Author Checklist
### Fixed Issues
section aboveTests
sectionOffline steps
sectionQA steps
sectiontoggleReport
and notonIconClick
)src/languages/*
files and using the translation methodSTYLE.md
) were followedAvatar
, I verified the components usingAvatar
are working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)
)Avatar
is modified, I verified thatAvatar
is working as expected in all cases)Design
label and/or tagged@Expensify/design
so the design team can review the changes.ScrollView
component to make it scrollable when more elements are added to the page.main
branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTest
steps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop