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

Handover to https://new.expensify.com/r/concierge is not seamless #51759

Closed
8 tasks
muttmuure opened this issue Oct 30, 2024 · 12 comments
Closed
8 tasks

Handover to https://new.expensify.com/r/concierge is not seamless #51759

muttmuure opened this issue Oct 30, 2024 · 12 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff

Comments

@muttmuure
Copy link
Contributor

muttmuure commented Oct 30, 2024

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:
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @muttmuure
Slack conversation (hyperlinked to channel name): https://expensify.slack.com/archives/C07NMDKEFMH/p1730316136748629?thread_ts=1730247644.696569&cid=C07NMDKEFMH

Action Performed:

Break down in numbered steps

Log into Expensify Classic
Navigate to https://new.expensify.com/r/concierge

Expected Result:

Describe what you think should've happened

Your chat with Concierge is loaded fully in a short time

Actual Result:

Describe what actually happened

The chat loads for a long time, and only the latest message is shown

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

None

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @RachCHopkins
@muttmuure muttmuure added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Oct 30, 2024
Copy link

melvin-bot bot commented Oct 30, 2024

Triggered auto assignment to @RachCHopkins (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.

@youssef-lr youssef-lr self-assigned this Oct 30, 2024
@youssef-lr youssef-lr added the Internal Requires API changes or must be handled by Expensify staff label Oct 30, 2024
@trjExpensify trjExpensify moved this to Product (CRITICAL) in [#whatsnext] #retain Nov 1, 2024
@melvin-bot melvin-bot bot added the Overdue label Nov 4, 2024
@youssef-lr
Copy link
Contributor

Auth is merged. I'll have the App PR ready by the time Auth deploys.

@melvin-bot melvin-bot bot removed the Overdue label Nov 4, 2024
@maddylewis maddylewis moved this from Product (CRITICAL) to Support (HIGH) in [#whatsnext] #retain Nov 4, 2024
@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Daily KSv2 Weekly KSv2 labels Nov 6, 2024
@melvin-bot melvin-bot bot changed the title Handover to https://new.expensify.com/r/concierge is not seamless [HOLD for payment 2024-11-20] Handover to https://new.expensify.com/r/concierge is not seamless Nov 13, 2024
Copy link

melvin-bot bot commented Nov 13, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Nov 13, 2024
Copy link

melvin-bot bot commented Nov 13, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.60-3 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-11-20. 🎊

Copy link

melvin-bot bot commented Nov 13, 2024

@youssef-lr @RachCHopkins @youssef-lr The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Nov 19, 2024
@youssef-lr
Copy link
Contributor

No payment is needed here, this has been worked on internally.

@youssef-lr youssef-lr changed the title [HOLD for payment 2024-11-20] Handover to https://new.expensify.com/r/concierge is not seamless Handover to https://new.expensify.com/r/concierge is not seamless Nov 19, 2024
@github-project-automation github-project-automation bot moved this from Support (HIGH) to Done in [#whatsnext] #retain Nov 19, 2024
@Ollyws
Copy link
Contributor

Ollyws commented Nov 25, 2024

@youssef-lr @RachCHopkins I'm due payment for the review of #52141, could we open this back up? Thanks.

@RachCHopkins RachCHopkins reopened this Nov 26, 2024
@RachCHopkins
Copy link
Contributor

@Ollyws I see you are paid via NewDot Manual Request, so no offer needed. But @youssef-lr do you know if this has a standard $250 pricetag?

@youssef-lr
Copy link
Contributor

This was a pretty simple PR so I think the payout is $125 here.

@RachCHopkins
Copy link
Contributor

Payment Summary:

  • Contributor+: @Ollyws to be paid $125 via NewDot manual request

No Upwork job

@Ollyws
Copy link
Contributor

Ollyws commented Nov 27, 2024

Requested in ND.

@garrettmknight
Copy link
Contributor

$250 approved for @Ollyws

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff
Projects
Status: DONE
Development

No branches or pull requests

5 participants