-
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
[$500] [Medium] TRIPPREVIEW is not showing the preview/info #45348
Comments
Triggered auto assignment to @johncschuster ( |
@johncschuster Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Given the urgency, I've opted to assign this to the engineers curating the |
Hmm, I thought that would double it. |
Reproduction steps are unclear, is there any video to demonstrate the issue clearly?! |
@b4s36t4 I see you requested in Slack to be added to the beta. I've just done that now. |
@johncschuster hitting issue as mentioned here #45647 |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@iwiznia, @johncschuster, @danieldoglas, @stitesExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Thanks for the heads up, @b4s36t4! I'll keep an eye on that issue as well. |
Hi @b4s36t4 can you try again? |
Not overdue, Melv |
@iwiznia, @johncschuster, @cristipaval, @stitesExpensify Whoops! This issue is 2 days overdue. Let's get this updated quick! |
I'm working on a PR to block creating trip rooms for the pre-paid bookings until the payment is confirmed. |
Not overdue. @cristipaval's working on a PR |
Spotnana finally replied to my questions, here |
The Auth PR is ready for review. |
Fix PR hit production. Closing |
Hello @cristipaval I am sorry for asking this |
@johncschuster could you please check the above comment? 🙏 |
@shubham1206agra can you help me understand what the compensation would be for? Would it be for reporting the issue, or did you contribute to the PR that led to the resolution? If the latter, can you point out where you contributed? That'll make it way easier to review given the length of this issue. Thank you! |
@johncschuster Did both, but there are a lot of places where we discussed this in Slack (some are in private DMs). One thread I found is in #45348 (comment) |
Thanks for your comment! I can see where you contributed via Slack (that thread was helpful!). Did you contribute to a PR, either as a contributor or as a reviewer? |
Not really. Since the fix was made internally. |
I also participated in that Slack thread, where we chatted about the steps to reproduce the issue. I think we usually do a partial pay when the C+ reviews proposals before we decide to take the issue internally. We also do this when we accept a proposal, the contributor starts working on it, and then we decide to close the issue or fix it internally. Other than the bug report in Slack, I can't tell concrete contributions from @shubham1206agra to the solution, @johncschuster, so I'm not sure if the #bug-zero team pays anything in these situations. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation: https://expensify.slack.com/archives/C05S5EV2JTX/p1720621730730619
Action Performed:
Expected Result:
Preview should load/show info
Actual Result:
Preview is not reliably loading/showing information
https://files.slack.com/files-pri/T047TPA624F-F07BMUQ00P8/screenshot_2024-07-10_at_7.58.02___pm.png
Screenshots/Videos
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @johncschusterThe text was updated successfully, but these errors were encountered: