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.

Sign up for GitHub

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

Jump to bottom

Update Summary Card for Orders #2045

Closed
rymorgan opened this issue Mar 27, 2017 · 5 comments
Closed

Update Summary Card for Orders #2045

rymorgan opened this issue Mar 27, 2017 · 5 comments
Assignees
@joykare
Milestone
Next

Comments

@rymorgan
Copy link
Contributor

rymorgan commented Mar 27, 2017 •
edited by joykare
Loading

Put summary into collapsable card.
Clean up layout of panel typography
Add order customer name and order number to top of card
Add status label

orders summary card

Annotated card

orders_summary_card-annotated
@rymorgan rymorgan added the UIX label Mar 27, 2017
@rymorgan rymorgan added this to the Next milestone Mar 27, 2017
@rymorgan rymorgan added the ready label Mar 27, 2017
@rymorgan rymorgan mentioned this issue Mar 27, 2017
Closed
10 tasks
@joykare joykare self-assigned this Mar 28, 2017
@joykare joykare added in progress and removed ready labels Mar 28, 2017
@joykare
Copy link
Contributor

joykare commented Mar 30, 2017

@rymorgan @mikemurray Could I just change this over to react? Changing the templates would be just as much code refactor.

@mikemurray
Copy link
Member

You are free to make anything and everything React!

@ghost ghost added review and removed in progress labels Apr 11, 2017
@joykare joykare mentioned this issue Apr 11, 2017
Merged
@rymorgan
Copy link
Contributor Author

Looking really good! A couple of small things to finish it up. See notes below.

2045-notes

@rymorgan
Copy link
Contributor Author

rymorgan commented Apr 11, 2017 •
edited
Loading

@mikemurray We should put this in the <SettingsCard> component in another PR. So if you can fix the gap we're good to go on this issue.

Here's the issue:
#2111

@mikemurray mikemurray mentioned this issue Apr 18, 2017
Closed
40 tasks
@brent-hoover
Copy link
Collaborator

Closed via #2105

@ghost ghost removed the review label Apr 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joykare joykare

Labels
None yet
Projects
None yet
Milestone
Next
Development

No branches or pull requests
4 participants
@mikemurray @brent-hoover @joykare @rymorgan