-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Android - Chat - Infinite loading spinner appears when adding PDF attachment #7267
Comments
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @mountiny ( |
Identified a potential culprit PR and left a comment there for involved developers if they are able to help: #4908 |
Submitted a regression fix: #7277
Maybe we should include a point dealing with PRs that introduce a migration |
That is a good point @kidroca. Thank you very much for digging into this! |
@kavimuru @mvtglobally The fix has been deployed to staging, would you be please able to re-test this? Thank you very much! 🙇 |
Triggered auto assignment to @michaelhaxhiu ( |
@michaelhaxhiu I made this issue I am not sure how exactly his contract works but as per this comment, Kidroca will track time working on that fix under this issue. @kidroca do we need to export this job and assign it to you on Upwork too? |
No, typically I reference the issue I was assigned and worked on in my hourly This task would take about an hour, and looks like a PR would be welcomed: react-native-documents/document-picker#526 (comment) |
@kidroca Sounds great then! Thank you for working on this! |
PR was merged and a new version (8) with the change is available. |
Thank you @kidroca! |
PR to update App's version is ready: #7338 |
Issue not reproducible during KI retests. (First week) |
Issue not reproducible during KI retests. (Second week) |
Issue not reproducible during KI retests. (Third week) |
What is the general rule for KI retests? At what point are we happy to close the issue? @mvtglobally Thank you :) |
@mountiny typically we close after third no-repro. |
@mvtglobally Ah! Thank you very much for explanation! In case of @kidroca, he works on hourly basis, therefore we are good to close this issue, they will be paid. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
PDF file should upload
Actual Result:
Infinite loading spinner appears and never gets added
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.30-0
Reproducible in staging?: Yes
Reproducible in production?: No
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
Bug5409742_20220116_103712__1_.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: