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

[Payment card / Subscription] refactor ADD_DEBIT_CARD_FORM Onyx key to be agnostic of debit vs credit in subscription context #43963

Closed
blimpich opened this issue Jun 18, 2024 · 6 comments
Assignees
Labels
Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review

Comments

@blimpich
Copy link
Contributor

blimpich commented Jun 18, 2024

Right now we're using the Onyx key ADD_DEBIT_CARD_FORM in the add-payment-card page for the new subscription page. This is a misnomer because the card can be either a debit or credit card. #43572 fixed this in the frontend, now we need to fix it in the backend.

Extra Context:
PR where it was introduced and discussion that initiated all this: #42771 (comment)
relevant slack thread: https://expensify.slack.com/archives/C036QM0SLJK/p1717606582669759
where we probably need to address it in web: https://github.com/Expensify/Web-Expensify/pull/42245/files#r1642211354
where App TODO is:

// TODO refactor ONYXKEYS.FORMS.ADD_DEBIT_CARD_FORM to ONYXKEYS.FORMS.ADD_CARD_FORM as a follow up

Slack thread keeping track of Callstack engineer who is working on the frontend portion of this: https://expensify.slack.com/archives/C036QM0SLJK/p1718659002462059

@blimpich blimpich self-assigned this Jun 18, 2024
@blimpich blimpich converted this from a draft issue Jun 18, 2024
@blimpich blimpich added Engineering Daily KSv2 Internal Requires API changes or must be handled by Expensify staff labels Jun 18, 2024
@blimpich blimpich changed the title [Payment card / Subscription] refactor ADD_DEBIT_CARD_FORM to be ADD_CARD_FORM where appropriate [Payment card / Subscription] refactor ADD_DEBIT_CARD_FORM Onyx key to be agnostic of debit vs credit in subscription context Jun 18, 2024
@melvin-bot melvin-bot bot added the Overdue label Jun 21, 2024
@blimpich
Copy link
Contributor Author

Waiting on #43572 to be merged I think before I can do this

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jun 21, 2024
@blimpich
Copy link
Contributor Author

still waiting

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jun 24, 2024
@blimpich
Copy link
Contributor Author

unblocked, will work on this today

@melvin-bot melvin-bot bot removed the Overdue label Jun 26, 2024
@blimpich blimpich added the Reviewing Has a PR in review label Jun 26, 2024
@trjExpensify
Copy link
Contributor

PR for this is on staging, will be closed once it hits prod!

Copy link

melvin-bot bot commented Jul 4, 2024

@blimpich Whoops! This issue is 2 days overdue. Let's get this updated quick!

@blimpich
Copy link
Contributor Author

blimpich commented Jul 4, 2024

on production now

@blimpich blimpich closed this as completed Jul 4, 2024
@github-project-automation github-project-automation bot moved this from Polish to Done in [#whatsnext] #wave-collect Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review
Projects
No open projects
Archived in project
Development

No branches or pull requests

2 participants