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

feat(payments): PAYMENTS-3143 Add Payment Methods to Account #1314

Merged

Conversation

jackosaurus
Copy link
Contributor

What?

Add Payment Methods to Account

  • new page called Payment Methods under Account
  • visibility of page to be controlled server side (experiment)
  • empty for now, more content coming in future pull requests

Tickets / Documentation

https://jira.bigcommerce.com/browse/PAYMENTS-3143

Add links to any relevant tickets and documentation.

Screenshots (if appropriate)

image

@bigcommerce/payments @junedkazi

- new page called Payment Methods under Account
- visibility of page to be controlled server side
@bigbot
Copy link

bigbot commented Jul 16, 2018

Autotagging @bigcommerce/storefront-team @davidchin

@jackosaurus jackosaurus requested a review from junedkazi July 17, 2018 01:05
@bc-biren
Copy link

💚

@jackosaurus jackosaurus changed the base branch from master to payment-methods-feature August 2, 2018 07:09
@@ -0,0 +1,7 @@
{{#partial "page"}}
{{> components/common/breadcrumbs breadcrumbs=breadcrumbs}}
Copy link
Contributor

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?

🍹

danieldelcore
danieldelcore previously approved these changes Aug 3, 2018
Copy link
Contributor

@danieldelcore danieldelcore left a 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>
Copy link
Contributor

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>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and here

@danieldelcore danieldelcore dismissed their stale review August 3, 2018 00:42

Sorry, just noticed that you're missing some translations

Copy link
Contributor

@danieldelcore danieldelcore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jackosaurus
Copy link
Contributor Author

Merging into feature branch

@jackosaurus jackosaurus merged commit ee57ead into bigcommerce:payment-methods-feature Aug 6, 2018
@jackosaurus jackosaurus deleted the payment-methods branch August 6, 2018 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants