-
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
Customer Order grid link issue. #1330
Comments
Watching football and coding so I made a PR to implement this fix. #1343 Thanks for the solutions. Cheers! |
Hi @hiteshvaghasiya83 . It looks like this was addressed in #1343, which was released as a part of Cornerstone 2.4.0: https://github.com/bigcommerce/cornerstone/releases/tag/2.4.0 . If this has addressed this issue, could you close it? |
Hi @hiteshvaghasiya83 . I'm going to close this issue because of the fix in #1343 . Please reopen if it appears in the future. |
Expected behavior
Customer My Account Page
Actual behavior
Steps to reproduce the behavior
Solution
Current Code
{{#if account_page '===' 'orders'}}<li class="navBar-item is-active">{{lang 'account.nav.orders'}}</li>{{else}}<li class="navBar-item"><a class="navBar-action" href="{{urls.account.orders.all}}">{{lang 'account.nav.orders'}}</a></li>{{/if}}
Solution Code
{{#if account_page '===' 'orders'}} {{#if page_type '===' 'account_order'}} <li class="navBar-item is-active"> <a class="navBar-action" href="{{urls.account.orders.all}}">{{lang 'account.nav.orders'}}</a> </li> {{else}} <li class="navBar-item is-active">{{lang 'account.nav.orders'}}</li> {{/if}} {{else}} <li class="navBar-item"> <a class="navBar-action" href="{{urls.account.orders.all}}">{{lang 'account.nav.orders'}}</a> </li> {{/if}}
The text was updated successfully, but these errors were encountered: