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

records: add a parameter for sorting records #64

Merged
merged 1 commit into from
Nov 21, 2019
Merged

records: add a parameter for sorting records #64

merged 1 commit into from
Nov 21, 2019

Conversation

sebdeleze
Copy link

  • Adds an optional parameter for sorting records in GetRecords method.
  • Imports TranslateModule for initializing translations in ng-core-tester.
  • Exports BsDropdownModule in CoreModule to use it in RecordModule.
  • Adds new translations.
  • Puts query params in URL if they are not present, in RecordSearchComponent.

Co-Authored-by: Sébastien Délèze sebastien.deleze@rero.ch

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?

@sebdeleze sebdeleze requested a review from jma November 21, 2019 14:29
* Adds an optional parameter for sorting records in GetRecords method.
* Imports TranslateModule for initializing translations in ng-core-tester.
* Exports BsDropdownModule in CoreModule to use it in RecordModule.
* Adds new translations.
* Puts query params in URL if they are not present, in RecordSearchComponent.

Co-Authored-by: Sébastien Délèze <sebastien.deleze@rero.ch>
@sebdeleze sebdeleze merged commit a07d7cb into rero:dev Nov 21, 2019
@sebdeleze sebdeleze deleted the sed-sort-param branch November 21, 2019 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants