-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[HOLD for payment 2024-06-11] [$500] Private notes - Hmm its not here page displayed when clicking on thumbnail #38835
Comments
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Triggered auto assignment to @chiragsalian ( |
@chiragsalian FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
Don't think this is blocker since markdown image is new feature and still WIP of polish cc: @kidroca |
Yup, I agree too. This does not feel blocker-worthy since it's a new feature. A simple resolution would be to revert the feature here - #37566 but I would prefer if move forward and tidy up this bug. |
edge case that I wouldn't consider much of a blocker, but what I would recommend is that it opens an attachment carousel that only includes attachments in the |
in my opinion the value of inline images way outweighs the downside of this bug, so I wouldn't want to see us revert that because of this |
ProposalPlease re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
What changes do you think we should make in order to solve the problem?
What alternative solutions did you explore? (Optional)
ResultScreen.Recording.2024-03-23.at.12.49.59.mov |
Job added to Upwork: https://www.upwork.com/jobs/~01c6ef0c33505c63ef |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra ( |
Triggered auto assignment to @MitchExpensify ( |
📣 @shubham1206agra 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
📣 @nkdengineer You have been assigned to this job! |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.78-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2024-06-11. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
@chiragsalian, @MitchExpensify, @shubham1206agra, @nkdengineer Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Payment summary:
|
paid and contract ended, @shubham1206agra 🙇 Do we need to add any new tests here? |
@MitchExpensify I don't think so |
@MitchExpensify Am I missing something here. Why the payout is decreased here? |
I did not receive any offers from Upwork. |
I saw the Upwork job created a $250 amount automatically so I was basing it off that. I will update if $500 is correct |
I think I got my wires crossed with this similar looking job on Upwork: https://www.upwork.com/nx/wm/workroom/37532304/details Sorry about that @shubham1206agra ! Rectifying my mistake here https://www.upwork.com/nx/wm/offer/102750200 |
@MitchExpensify Accepted offer |
Paid and contract ended, thanks! Do we need BZ steps here? |
No |
@MitchExpensify anything pending here or can we close this out. |
Sorry @MitchExpensify It seems I wasn't paid here, could you please reopen the issue to help with this? TIA |
Oh weird, it looks like the automation flagged you as due to be paid via New Expensify so I thought we were all done:
Here's a new offer to resolve this https://www.upwork.com/nx/wm/offer/102961478 |
Paid and contract ended, thanks for the catch @nkdengineer |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.56-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): natnael.expensify+3@gmail.com
Issue reported by: Applause - Internal Team
Issue found when executing PR #37566
Action Performed:
Expected Result:
Actual Result:
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6423400_1711129594737.Screen_Recording_2024-03-22_at_8.44.37_in_the_evening.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @MitchExpensifyThe text was updated successfully, but these errors were encountered: