[HOLD for payment 2024-11-29] [$250] Deeplink - Duplicate concierge chat is displayed on LHN when deeplinking to concierge

[lanitochka17]

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: 9.0.2-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Issue found when executing PR #44063

Action Performed:

1. Sign out from ND
2. Paste this link to the browser tab https://staging.new.expensify.com/concierge
3. Press Enter key
4. Enter a new user email address in the email field
5. Click Continue button
6. Click the Join button
7. While staying in the onboarding process, observe the LHN

Expected Result:

Only one concierge chat is displayed on the LHN

Actual Result:

Duplicate concierge chat is briefly displayed on the LHN

Workaround:

Unknown

Platforms:

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

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6524808_1719378689822.2024-06-26_08_06_58.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01761bb8fe9958d8df
• Upwork Job ID: 1808128563862616294
• Last Price Increase: 2024-10-16

Issue Owner

Current Issue Owner: @getusha

[melvin-bot]

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

[lanitochka17]

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

[lanitochka17]

We think that this bug might be related to #vip-vsp

[melvin-bot]

@sonialiap Huh... This is 4 days overdue. Who can take care of this?

[melvin-bot]

Job added to Upwork: https://www.upwork.com/jobs/~01761bb8fe9958d8df

[melvin-bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha (External)

[jnowakow]

Hello, I'm from SoftwareMansion, an expert agency, and I'm going to work on this one

[jnowakow]

It's strange, I'm able to reproduce this issue on staging but when I try to do so on local build it works correctly:

repro2-720.mov

[getusha]

Same, not able to reproduce this on local build.

[melvin-bot]

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

[melvin-bot]

@sonialiap, @getusha Huh... This is 4 days overdue. Who can take care of this?

[melvin-bot]

@sonialiap @getusha this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

[melvin-bot]

@sonialiap, @getusha 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

[sonialiap]

@jnowakow @getusha I'm not sure how local build is created (not an engineer 😅), is the local build based on staging or is it possible for staging to be different from a local build? Maybe the builds have to be renewed?

[melvin-bot]

@sonialiap, @getusha 10 days overdue. Is anyone even seeing these? Hello?

[melvin-bot]

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

[melvin-bot]

@sonialiap, @getusha 12 days overdue now... This issue's end is nigh!

[mvtglobally]

Issue not reproducible during KI retests. (First week)

[melvin-bot]

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

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.65-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Fixing duplicate Concierge chat #51794

If no regressions arise, payment will be issued on 2024-11-29. 🎊

For reference, here are some details about the assignees on this issue:

• @getusha requires payment through NewDot Manual Requests
• @ugogiordano requires payment (Needs manual offer from BZ)

[melvin-bot]

@getusha @sonialiap @getusha 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]

Payment Summary

Upwork Job

• Reviewer: @getusha owed $250 via NewDot
• ROLE: @ugogiordano paid $(AMOUNT) via Upwork (LINK)

BugZero Checklist (@sonialiap)

• I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
• I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants/1808128563862616294/hired)
• I have paid out the Upwork contracts or cancelled the ones that are incorrect
• I have verified the payment summary above is correct

[sonialiap]

Payment summary:

• @getusha $250 via NewDot - please complete the checklist
• @ugogiordano $250 - offer sent in upwork - paid ✔️

[ugogiordano]

@sonialiap, can I get the offer on the first "Upwork Profile for payment" you can see on my bio? I have attached the VAT ID to it.

[sonialiap]

@ugogiordano the offer should be sent to Rosaria V., is that correct?

[ugogiordano]

@ugogiordano the offer should be sent to Rosaria V., is that correct?

Yes, correct, thank you so much!

[sonialiap]

@ugogiordano offer paid!

https://www.upwork.com/nx/wm/offer/105173644

[melvin-bot]

@sonialiap, @getusha, @aldo-expensify, @ugogiordano Huh... This is 4 days overdue. Who can take care of this?

[sonialiap]

@getusha bumping for the checklist :)

[melvin-bot]

@sonialiap, @getusha, @aldo-expensify, @ugogiordano Still overdue 6 days?! Let's take care of this!

[getusha]

BugZero Checklist:

• [Contributor] Classify the bug:

Bug classification

Source of bug:

• 1a. Result of the original design (eg. a case wasn't considered)
• 1b. Mistake during implementation
• 1c. Backend bug
• 1z. Other:

Where bug was reported:

• 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
• 2b. Reported on staging (eg. found during regression or PR testing)
• 2d. Reported on a PR
• 2z. Other:

Who reported the bug:

• 3a. Expensify user
• 3b. Expensify employee
• 3c. Contributor
• 3d. QA
• 3z. Other:

• [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.

  Link to comment: N/a this was fixed before [HOLD for payment 2023-06-02] [HOLD for payment 2023-06-01] [$1000] Duplicate Concierge chat created when user sign in with https://new.expensify.com/concierge link #17884

• [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.

  Link to discussion: N/a

• [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

We already have a regression test created for this issue #17884

[sonialiap]

Thanks!

[garrettmknight]

$250 approved for @getusha