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

Fix admin user order history table #2226

Merged
merged 1 commit into from
Sep 20, 2017

Conversation

Sinetheta
Copy link
Contributor

A small typo results in an error when viewing this table if the user has
any orders.

I don't see any backend request specs and I can't get a solidus test app running, so I appollogise but I'm submitting this one blind ;)

A small typo results in an error when viewing this table if the user has
any orders.
@jhawthorn jhawthorn added this to the 2.4.0 milestone Sep 18, 2017
Copy link
Contributor

@jhawthorn jhawthorn left a comment

Choose a reason for hiding this comment

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

Nice catch. We should make sure to add a spec for this page soon.

Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks

@mamhoff mamhoff merged commit 3f485cf into solidusio:master Sep 20, 2017
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