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

Payments tab for orders page #1481

Closed
dmdj03 opened this issue Sep 29, 2014 · 6 comments · Fixed by #1505
Closed

Payments tab for orders page #1481

dmdj03 opened this issue Sep 29, 2014 · 6 comments · Fixed by #1505

Comments

@dmdj03
Copy link

dmdj03 commented Sep 29, 2014

Instead of having separate sub-tabs in the side navigation for orders and transactions, there will be one payments tab with a toggle between orders and transactions. Since orders will become the default, we want to emphasize a closer relationship between orders and transactions—orders being the higher level view, using a different layout (card view) than the transactions table.

The red chain/x icon indicates an unlinked transaction.

screen shot 2014-09-29 at 10 48 43 am

screen shot 2014-09-29 at 10 48 54 am

screen shot 2014-09-29 at 10 49 04 am

screen shot 2014-09-29 at 10 59 11 am

@dmdj03
Copy link
Author

dmdj03 commented Oct 10, 2014

Indicating "overdue Orders" past 30 days on the payments page:

screen shot 2014-10-09 at 5 15 49 pm

@dmdj03
Copy link
Author

dmdj03 commented Oct 10, 2014

Indicating unlinked transactions:
screen shot 2014-10-09 at 5 18 43 pm

@kyungmin
Copy link
Contributor

@dmdj03 Can you update this with latest mocks? Looks like "Debit a card" and "Credit a bank account" buttons are missing, as well as "Import payouts" link.

@kyungmin
Copy link
Contributor

@dmdj03
Copy link
Author

dmdj03 commented Oct 22, 2014

@dmdj03
Copy link
Author

dmdj03 commented Oct 22, 2014

Here's the update to the icons for the toggle button
screen shot 2014-10-22 at 10 59 11 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants