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

[$500] [Medium] TRIPPREVIEW is not showing the preview/info #45348

Closed
kevinksullivan opened this issue Jul 12, 2024 · 97 comments
Closed

[$500] [Medium] TRIPPREVIEW is not showing the preview/info #45348

kevinksullivan opened this issue Jul 12, 2024 · 97 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review

Comments

@kevinksullivan
Copy link
Contributor

kevinksullivan commented Jul 12, 2024

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:

  1. Book a trip in spotnana
  2. Navigate to the workspace chat

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

image

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~014f530c69599f191f
  • Upwork Job ID: 1812949990970484758
  • Last Price Increase: 2024-09-30
Issue OwnerCurrent Issue Owner: @johncschuster
@kevinksullivan kevinksullivan added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jul 12, 2024
Copy link

melvin-bot bot commented Jul 12, 2024

Triggered auto assignment to @johncschuster (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added the Overdue label Jul 15, 2024
Copy link

melvin-bot bot commented Jul 15, 2024

@johncschuster Whoops! This issue is 2 days overdue. Let's get this updated quick!

@johncschuster
Copy link
Contributor

johncschuster commented Jul 15, 2024

Given the urgency, I've opted to assign this to the engineers curating the #travel-gbta-july-22-2024 project. Please reassign as needed!

@stitesExpensify stitesExpensify added External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors labels Jul 15, 2024
@melvin-bot melvin-bot bot changed the title [CRITICAL] TRIPPREVIEW is not showing the preview/info [$250] [CRITICAL] TRIPPREVIEW is not showing the preview/info Jul 15, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 18, 2024
@johncschuster
Copy link
Contributor

@twisterdotcom
Copy link
Contributor

Hmm, I thought that would double it.

@b4s36t4
Copy link
Contributor

b4s36t4 commented Jul 19, 2024

Reproduction steps are unclear, is there any video to demonstrate the issue clearly?!

@johncschuster
Copy link
Contributor

@b4s36t4 I see you requested in Slack to be added to the beta. I've just done that now.

@b4s36t4
Copy link
Contributor

b4s36t4 commented Jul 19, 2024

@johncschuster hitting issue as mentioned here #45647

@melvin-bot melvin-bot bot added the Overdue label Jul 22, 2024
Copy link

melvin-bot bot commented Jul 22, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

Copy link

melvin-bot bot commented Jul 22, 2024

@iwiznia, @johncschuster, @danieldoglas, @stitesExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@johncschuster
Copy link
Contributor

Thanks for the heads up, @b4s36t4! I'll keep an eye on that issue as well.

@melvin-bot melvin-bot bot removed the Overdue label Jul 22, 2024
@kevinksullivan
Copy link
Contributor Author

Hi @b4s36t4 can you try again?

@cristipaval
Copy link
Contributor

Not overdue, Melv

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Nov 28, 2024
Copy link

melvin-bot bot commented Dec 2, 2024

@iwiznia, @johncschuster, @cristipaval, @stitesExpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

@cristipaval
Copy link
Contributor

I'm working on a PR to block creating trip rooms for the pre-paid bookings until the payment is confirmed.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Dec 2, 2024
@johncschuster
Copy link
Contributor

Not overdue. @cristipaval's working on a PR

@melvin-bot melvin-bot bot removed the Overdue label Dec 4, 2024
@cristipaval
Copy link
Contributor

Spotnana finally replied to my questions, here

@melvin-bot melvin-bot bot added the Overdue label Dec 9, 2024
@iwiznia iwiznia removed their assignment Dec 9, 2024
@melvin-bot melvin-bot bot removed the Overdue label Dec 9, 2024
@cristipaval
Copy link
Contributor

The Auth PR is ready for review.

@cristipaval cristipaval added the Reviewing Has a PR in review label Dec 10, 2024
@cristipaval
Copy link
Contributor

Fix PR hit production. Closing

@github-project-automation github-project-automation bot moved this from Medium - NewDot MVP - Feb '25 to Done in [#whatsnext] #travel 2025-02-01 - External Commit Dec 12, 2024
@shubham1206agra
Copy link
Contributor

Hello @cristipaval

I am sorry for asking this
But am I eligible for compensation here?
Since I originally reported the issue https://expensify.slack.com/archives/C05S5EV2JTX/p1720621730730619
And I tried to drive this issue here https://expensify.slack.com/archives/C05S5EV2JTX/p1725546707583219

@cristipaval cristipaval reopened this Dec 12, 2024
@github-project-automation github-project-automation bot moved this from Done to Critical - Feb '25 Earnings in [#whatsnext] #travel 2025-02-01 - External Commit Dec 12, 2024
@cristipaval
Copy link
Contributor

@johncschuster could you please check the above comment? 🙏

@johncschuster
Copy link
Contributor

@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!

@shubham1206agra
Copy link
Contributor

@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)

@johncschuster
Copy link
Contributor

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?

@shubham1206agra
Copy link
Contributor

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.

@stitesExpensify stitesExpensify moved this from Critical - Feb '25 Earnings to Medium - NewDot MVP - Feb '25 in [#whatsnext] #travel 2025-02-01 - External Commit Dec 17, 2024
@cristipaval
Copy link
Contributor

cristipaval commented Dec 17, 2024

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.

@github-project-automation github-project-automation bot moved this from Medium - NewDot MVP - Feb '25 to Done in [#whatsnext] #travel 2025-02-01 - External Commit Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review
Development

No branches or pull requests