FAQ | This is a LIVE service | Changelog

Skip to content

add sorting to the asset list

Dr Rich Wareham requested to merge github/fork/rjw57/add-sorting-and-filtering into master

Add sorting to the asset list. Filtering would also be possible but there is no UI design for it yet so I've left it for another story. This is done with two commits:

e32351bc adds a new action, getAssets, which fetches a new list of assets from the API with a query specifying search, sorting and filtering. When the request succeeds, the redux state is updated to record the query so that UI components can react.

78cdb60 adds some UI components which react to this change by re-factoring the AssetTable body and header into two components and making use of the new TableSortLabel component from MUI v1 to render a nice header.

Update: After merging current master, I found that the asset list was not correctly refreshed after asset creation. a94f9553 fixes that and 8ec87dad ensures that the user does not lose their current sort settings by editing an asset. Finally, e792424f corrects the logic used to detect an asset list fetch by examining the assets.fetchedAt value in the redux store state.

Closes #21 (closed)

Merge request reports