-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add UI for collections, fix local graphs and improve tables #404
Conversation
Passing run #392 ↗︎
Details:
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. |
Deployed this to dev, one note is that this seems to require some migration somewhere to work out. Will investigate tomorrow. |
4d569b1
to
1b42f68
Compare
Going to merge this as-is as its been hanging around for ages, everything now works but I've removed collection blocks as they were requiring 100s of lines of UI duplication to get working -- better to add as a blocks refactor. |
1b42f68
to
5a89fe3
Compare
- List collection membership in sample/cell pages - Add collections API connections and associated setters (move up) - Add collection type to resources - Add collections-related fields to store - Add ability to insert new item straight into an existing collection - Add simplified collections views - Add collection select component - Re-add sample table for collections - Make the collection sample table work by reworking the store - Add UI components for formatting collection names - Tweak formatted item names/refcode components to enable multiple on same line
…omponent - Also fix sizing type mismatch in `UserBubble`
- Add cypress and log dirs to gitignore - Refactor babel to only strip console in production
- Fix graph tests - Fix e2e tests again - One more tweak to e2e - Fix trailing slash
- Add collection nodes and edges to graph view - Fix local graph by properly dealing with bi-directional relationships - Add collection-level graphs - Fix local graph tests when using collections
5a89fe3
to
21af81b
Compare
- Add special node type for the centre of a local graph - Default to graph view on item page - Tweak user bubbles
9862dc9
to
b3bd5b8
Compare
Hmmm, still some spurious e2e failures, will investigate on dev later before merging |
- Remove mention of collection blocks in UI until later refactor
- Update cypress tests - Update starting material UI with collections list
1d7f198
to
a9d9e8f
Compare
a9d9e8f
to
d87faa5
Compare
This is finally ready -- still need to work on #423 but this is enough for now |
This PR supersedes #292 by taking out the basic collections UI functionality of
Blocked by #403.
Buttons for adding old items to old collections(opened as Missing UI for adding/removing an existing item to a collection #423)