-
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
[HOLD for payment 2024-12-19] [$250] [HOLD for payment 2024-12-16] Incorrect default feed name in feed selector #53470
Comments
Triggered auto assignment to @jliexpensify ( |
Hi, I'm Viktoryia from Callstack - expert contributor group - and I would like to work on this issue. |
Small PR has been opened for the review: #53536 |
TY! |
PR is on staging ♻️ |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.72-1 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-12-16. 🎊 For reference, here are some details about the assignees on this issue:
|
@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] |
Job added to Upwork: https://www.upwork.com/jobs/~021866248038611712453 |
Current assignee @allgandalf is eligible for the External assigner, not assigning anyone new. |
Oh wait, just checking - @allgandalf will you be paid for this job specifically, or are you bundling this job into a larger payment? |
We will bundle the payments |
Ok cool, i shall close the Upworks job then - thanks Vit |
LMK if anyone needs a payment summary. Upworks job is closed now. |
@allgandalf Can you add the checklist and then we can close |
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.74-8 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-12-19. 🎊 For reference, here are some details about the assignees on this issue:
|
@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] |
No payment needed |
BugZero Checklist:
Bug classificationSource of bug:
Where bug was reported:
Who reported the bug:
No regression test, this issue is part of project, so tests will be added at the end of project Regression Test Proposal Template
Regression Test ProposalPrecondition:Test:Do we agree 👍 or 👎 |
Thanks @allgandalf! |
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.Issue Owner
Current Issue Owner: @Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @jliexpensifyThe text was updated successfully, but these errors were encountered: