Skip to content
This repository has been archived by the owner on May 14, 2022. It is now read-only.

Add DataTables for Folder View #1218

Merged
merged 2 commits into from
Apr 26, 2017
Merged

Add DataTables for Folder View #1218

merged 2 commits into from
Apr 26, 2017

Conversation

tpendragon
Copy link
Contributor

Closes #1188

@tpendragon
Copy link
Contributor Author

Coverage for this isn't gonna be done - to fix it I'm going to have to get EphemeraFolderController to inherit from HyraxController, which opens up a big can of worms with expected schema properties and a conflict with single-valued property behavior. So, need to spin on this a bit. May have a supporting PR for redoing single-valued properties first.

@coveralls
Copy link

coveralls commented Apr 25, 2017 •
edited
Loading

Coverage Status

Coverage remained the same at 100.0% when pulling 96f02ce on datatables into 5d2706a on master.

@tpendragon
Copy link
Contributor Author

Guess I was wrong about coverage

@escowles escowles merged commit f0365d1 into master Apr 26, 2017
@escowles escowles deleted the datatables branch April 26, 2017 18:00
@escowles escowles added the LAE label Sep 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
LAE
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tpendragon @coveralls @escowles