Closed MitchExpensify closed 1 month ago
Triggered auto assignment to @Christinadobrzyn (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.
Update the payment card icon in the Subscription tab
The background color is not correct. https://github.com/Expensify/App/blob/54999880458a4c18204a56e37e1d4952a68b3b5a/src/styles/index.ts#L2876
Update it to theme.buttonDefaultBG
. Also update the fill={theme.text}
to fill={theme.icon}
here.
https://github.com/Expensify/App/assets/85894871/c33c723f-e8b6-42d5-bfc4-c690d909e78a
Going to export this at $125 as it's a simple icon change.
Job added to Upwork: https://www.upwork.com/jobs/~01131af4be64782018
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (External
)
Upwork job price has been updated to $125
I didn't understand the issue at all from the description. What needs to be changed @trjExpensify?
This icon is too dark:
@Krishna2323's proposal looks good to me.
:ribbon: :eyes: :ribbon: C+ reviewed
Current assignee @blimpich is eligible for the choreEngineerContributorManagement assigner, not assigning anyone new.
๐ฃ @Krishna2323 ๐ An offer has been automatically sent to your Upwork account for the Contributor role ๐ Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review ๐งโ๐ป Keep in mind: Code of Conduct | Contributing ๐
@parasharrajat, PR ready for review.
PR in staging - getting close! https://github.com/Expensify/App/pull/45219
Reviewing
label has been removed, please complete the "BugZero Checklist".
The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.7-8 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2024-07-24. :confetti_ball:
For reference, here are some details about the assignees on this issue:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
Preparing for payment
Payouts due:
@parasharrajat do we need a regression test for this?
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
Do you agree ๐ or ๐ ?
Payment day -
Closing this out since @parasharrajat will be paid through NewDot.
Payment requested as per https://github.com/Expensify/App/issues/45123#issuecomment-2246066127
$125 approved for @parasharrajat
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: v9.0.5-10 Reproducible in staging?: Staging Reproducible in production?: If this was caught during regression testing, add the test name, ID and link from TestRail: NA Email or phone of affected tester (no customers): Logs: NA Expensify/Expensify Issue URL: NA Issue reported by: mitch@expebsify.com Slack conversation: Internal: https://expensify.slack.com/archives/C036QM0SLJK/p1720560965872139?thread_ts=1720544914.651789&cid=C036QM0SLJK
Action Performed:
Expected Result:
New Feature, the card icon in the Payment section look like this:
Actual Result:
The card icon is darker than expected:
Workaround:
Ignore it
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Added above
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @Christinadobrzyn