App icon indicating copy to clipboard operation
App copied to clipboard

[$250] [HOLD for payment 2024-12-16] Incorrect default feed name in feed selector

Open joekaufmanexpensify opened this issue 1 year ago • 12 comments

Coming from here, when you first add a card feed, the default feed name is %Bank% cards. You can then edit this name to be whatever you want. The default feed name should appear consistently everywhere (the feed itself, feed settings, feed selector, etc.) It currently does not appear correctly on the feed selector.

image - 2024-12-03T090545 016

Issue OwnerCurrent Issue Owner: @
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021866248038611712453
  • Upwork Job ID: 1866248038611712453
  • Last Price Increase: 2024-12-09

joekaufmanexpensify avatar Dec 03 '24 14:12 joekaufmanexpensify

Triggered auto assignment to @jliexpensify (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[bot] avatar Dec 03 '24 14:12 melvin-bot[bot]

Hi, I'm Viktoryia from Callstack - expert contributor group - and I would like to work on this issue.

VickyStash avatar Dec 03 '24 14:12 VickyStash

Small PR has been opened for the review: https://github.com/Expensify/App/pull/53536

VickyStash avatar Dec 04 '24 10:12 VickyStash

TY!

joekaufmanexpensify avatar Dec 04 '24 13:12 joekaufmanexpensify

PR is on staging ♻️

allgandalf avatar Dec 06 '24 10:12 allgandalf

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

melvin-bot[bot] avatar Dec 09 '24 16:12 melvin-bot[bot]

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

  • https://github.com/Expensify/App/pull/53536

If no regressions arise, payment will be issued on 2024-12-16. :confetti_ball:

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

  • @VickyStash does not require payment (Contractor)
  • @allgandalf requires payment (Needs manual offer from BZ)

melvin-bot[bot] avatar Dec 09 '24 16:12 melvin-bot[bot]

@allgandalf @jliexpensify @allgandalf 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[bot] avatar Dec 09 '24 16:12 melvin-bot[bot]

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

melvin-bot[bot] avatar Dec 09 '24 22:12 melvin-bot[bot]

Current assignee @allgandalf is eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] avatar Dec 09 '24 22:12 melvin-bot[bot]

Oh wait, just checking - @allgandalf will you be paid for this job specifically, or are you bundling this job into a larger payment?

jliexpensify avatar Dec 09 '24 22:12 jliexpensify

We will bundle the payments

mountiny avatar Dec 10 '24 12:12 mountiny

Ok cool, i shall close the Upworks job then - thanks Vit

jliexpensify avatar Dec 10 '24 20:12 jliexpensify

LMK if anyone needs a payment summary. Upworks job is closed now.

jliexpensify avatar Dec 10 '24 20:12 jliexpensify

@allgandalf Can you add the checklist and then we can close

mountiny avatar Dec 11 '24 12:12 mountiny

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

  • https://github.com/Expensify/App/pull/53705

If no regressions arise, payment will be issued on 2024-12-19. :confetti_ball:

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

  • @VickyStash does not require payment (Contractor)
  • @allgandalf requires payment (Needs manual offer from BZ)

melvin-bot[bot] avatar Dec 12 '24 01:12 melvin-bot[bot]

@allgandalf @jliexpensify @allgandalf 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[bot] avatar Dec 12 '24 01:12 melvin-bot[bot]

No payment needed

jliexpensify avatar Dec 12 '24 02:12 jliexpensify

BugZero Checklist:

  • [x] [Contributor] Classify the bug:
Bug classification

Source of bug:

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

Where bug was reported:

  • [x] 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
  • [x] 3b. Expensify employee
  • [ ] 3c. Contributor
  • [ ] 3d. QA
  • [ ] 3z. Other:
  • [x] [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: https://github.com/Expensify/App/pull/49928/files#r1881678339

  • [x] [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.

No regression test, this issue is part of project, so tests will be added at the end of project

Regression Test Proposal Template
  • [ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.

    Link to issue:

Regression Test Proposal

Precondition:

Test:

Do we agree 👍 or 👎

allgandalf avatar Dec 12 '24 09:12 allgandalf

Thanks @allgandalf!

jliexpensify avatar Dec 12 '24 23:12 jliexpensify