-
Notifications
You must be signed in to change notification settings - Fork 9
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
[Enhancement kbss-cvut/termit-ui#520] Change record filtering #313
[Enhancement kbss-cvut/termit-ui#520] Change record filtering #313
Conversation
…ry detailed endpoint allowing result filtering.
…history query Query filters were using wrong variables, and removed unnecessary optional block.
…d vocabulary content detailed history endpoint
…ring vocabulary content changes by change type.
…ocabulary content endpoint return results when no filter is specified.
…on to ChangeRecordDao
…ulary & term history endpoints
…yId to return optional URI instead of throwing when term is not found (not in a vocabulary).
… VocabularyDaoTest to ChangeRecordDaoTest.
…ords to change context
… interface and add documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of minor comments. The query method in ChangeRecordDao
looks horrible, but in this case, I agree with you that splitting it into smaller methods would make it probably less readable...
src/main/java/cz/cvut/kbss/termit/rest/VocabularyController.java
Outdated
Show resolved
Hide resolved
src/test/java/cz/cvut/kbss/termit/persistence/dao/changetracking/ChangeRecordDaoTest.java
Outdated
Show resolved
Hide resolved
…on in TermController & add constructors for ChangeRecordFilterDto
…s with Utils#timestamp
736dc83
to
ab07485
Compare
# Conflicts: # src/main/java/cz/cvut/kbss/termit/persistence/dao/VocabularyDao.java # src/test/java/cz/cvut/kbss/termit/persistence/dao/VocabularyDaoTest.java # src/test/java/cz/cvut/kbss/termit/rest/VocabularyControllerTest.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, works nicely!
resolves kbss-cvut/termit-ui#520
Implements filtering in ChangeRecordDao.
Several tests required modification because change records were stored in the vocabulary context instead of the change context.