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

Update Data Fetching in Order Dashboard Table #2768

Closed
impactmass opened this issue Aug 30, 2017 · 0 comments
Closed

Update Data Fetching in Order Dashboard Table #2768

impactmass opened this issue Aug 30, 2017 · 0 comments
Assignees
Labels
chore For issues that describe a task that does not affect the release, such as tests or CI configuration

Comments

@impactmass
Copy link
Contributor

impactmass commented Aug 30, 2017

Order Dashboard Table is currently using ReactTable's way of passing publication/collection as opposed to passing it the actual data

This is to change that approach to passing the data to the table using the otherData prop as explained here #2631 (to follow similar way it's done in Accounts dashboard).

After this is done, ensure that all current parts of the table works as is (i.e the filters, search etc).
Also, when this is done, we should be able to easily achieve the last bit of the search requirements (#2355):

If there is exactly one order with matching customer information, the matching orderId should be opened in the action view panel if the user has permission to view that order.

@aaronjudd aaronjudd added chore For issues that describe a task that does not affect the release, such as tests or CI configuration and removed chore For issues that describe a task that does not affect the release, such as tests or CI configuration labels Sep 11, 2017
@joykare joykare self-assigned this Sep 26, 2017
@kieha kieha closed this as completed Sep 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore For issues that describe a task that does not affect the release, such as tests or CI configuration
Projects
None yet
Development

No branches or pull requests

4 participants