-
Notifications
You must be signed in to change notification settings - Fork 613
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
feat(payments): PAYMENTS-3143 Add Payment Methods to Account #1314
feat(payments): PAYMENTS-3143 Add Payment Methods to Account #1314
Conversation
- new page called Payment Methods under Account - visibility of page to be controlled server side
Autotagging @bigcommerce/storefront-team @davidchin |
💚 |
@@ -0,0 +1,7 @@ | |||
{{#partial "page"}} | |||
{{> components/common/breadcrumbs breadcrumbs=breadcrumbs}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indentation is a bit funny here, would you mind just aligning that import with the h2 tag?
🍹
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
|
||
{{#if settings.show_payment_methods}} | ||
{{#if account_page '===' 'payment_methods'}} | ||
<li class="navBar-item is-active">Payment Methods</li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing translation here
<li class="navBar-item is-active">Payment Methods</li> | ||
{{else}} | ||
<li class="navBar-item"> | ||
<a class="navBar-action" href="/account.php?action=payment_methods">Payment Methods</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and here
Sorry, just noticed that you're missing some translations
791f99c
to
7e8accf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Merging into feature branch |
What?
Add Payment Methods to Account
Tickets / Documentation
https://jira.bigcommerce.com/browse/PAYMENTS-3143
Add links to any relevant tickets and documentation.
Screenshots (if appropriate)
@bigcommerce/payments @junedkazi