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

call debounce only when it's required #4777

Merged
merged 1 commit into from
Aug 23, 2021

Conversation

aman-atg
Copy link
Contributor

Details

#4267 (comment)

Fixed Issues

$ #4267

Tests || QA Steps

  1. Login with two devices
  2. Open chat room for both of them

Use one of the account

  1. Type your message
  2. Submit/Send the message.
  3. It will NOT show *User* is typing after the message is sent.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Web.mov

Mobile Web

mWeb.mov

Desktop

Desktop.mov

iOS

IOS.mov

Android

Android.mov

@aman-atg aman-atg requested a review from a team as a code owner August 21, 2021 10:17
@MelvinBot MelvinBot requested review from tgolen and removed request for a team August 21, 2021 10:17
@tgolen
Copy link
Contributor

tgolen commented Aug 23, 2021

@aman-atg please always assign the PR to yourself :) I'll assign it this time for you

@tgolen tgolen merged commit f005e72 into Expensify:main Aug 23, 2021
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@aman-atg
Copy link
Contributor Author

Thanks @tgolen .
And I think it's not possible for external contributors to assign themselves.
Please correct me if that's not true.

@tgolen
Copy link
Contributor

tgolen commented Aug 23, 2021 via email

@aman-atg aman-atg removed their assignment Aug 23, 2021
@aman-atg
Copy link
Contributor Author

Screenshot 2021-08-23 at 8 33 47 PM

Yup, no option to assign myself.

@tgolen
Copy link
Contributor

tgolen commented Aug 23, 2021 via email

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.0.88-2 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@botify
Copy link

botify commented Sep 1, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-09-08. 🎊

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.

4 participants