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

document: filter results by org in admin view #208

Merged
merged 1 commit into from
Apr 24, 2020

Conversation

AoNoOokami
Copy link
Contributor

@AoNoOokami AoNoOokami commented Mar 17, 2020

Co-Authored-by: Alicia Zangger alicia.zangger@rero.ch
Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch

Why are you opening this PR?

https://tree.taiga.io/project/rero21-reroils/task/1377?kanban-status=1224895

How to test?

Needs rero/rero-ils#852 and last version of ng-core.

Public search

  1. Go to documents brief view
  2. You should have an organisation facet in global view
  3. You should have a library facet in organisation views.

Admin

  1. Go to documents brief view
  2. The results should be filtered by current organisation by default (test with different users).
  3. You should have a hierarchical facet (organisation + libraries)

Code review check list

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

@AoNoOokami AoNoOokami self-assigned this Mar 17, 2020
@AoNoOokami AoNoOokami force-pushed the zaa-#1377-filter-results-by-org branch 4 times, most recently from e0e25bc to 91f0a3a Compare March 23, 2020 07:37
@AoNoOokami AoNoOokami force-pushed the zaa-#1377-filter-results-by-org branch from 91f0a3a to 5dd8139 Compare March 25, 2020 08:05
@AoNoOokami AoNoOokami force-pushed the zaa-#1377-filter-results-by-org branch from 5dd8139 to e64286a Compare April 6, 2020 15:12
@AoNoOokami AoNoOokami marked this pull request as ready for review April 7, 2020 12:23
@AoNoOokami AoNoOokami force-pushed the zaa-#1377-filter-results-by-org branch from e64286a to ad42c61 Compare April 14, 2020 14:31
projects/admin/src/app/service/menu.service.ts Outdated Show resolved Hide resolved
projects/admin/src/app/service/menu.service.ts Outdated Show resolved Hide resolved
@AoNoOokami AoNoOokami force-pushed the zaa-#1377-filter-results-by-org branch 2 times, most recently from 53f3496 to bbdada0 Compare April 15, 2020 07:33
Copy link

@sebdeleze sebdeleze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some doc blocks are not well formatted (https://github.com/rero/rero-ils-ui/pull/208/files#diff-bf02d3d46f102054384e43e8deae3686R120, one empty line after description and @param and @return must be grouped), but it doesn't matter.

@AoNoOokami
Copy link
Contributor Author

Some doc blocks are not well formatted (https://github.com/rero/rero-ils-ui/pull/208/files#diff-bf02d3d46f102054384e43e8deae3686R120, one empty line after description and @param and @return must be grouped), but it doesn't matter.

Ok, corrected. There was a typo in the commit message anyway.

@AoNoOokami
Copy link
Contributor Author

AoNoOokami commented Apr 20, 2020

Needs ng-core publication to pass the CI: rero/ng-core#174

* Fixes display of organisation facet in document brief view.
* Modifies links to documents brief view to include current organisation as a default parameter.
* Closes rero#140.
* Removes useless code.
* Implements results and suggestions filtering for search in main bar.
* Updates @rero/ng-core version for the latest one.

Co-Authored-by: Alicia Zangger <alicia.zangger@rero.ch>
Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@AoNoOokami AoNoOokami force-pushed the zaa-#1377-filter-results-by-org branch from 38f2f4c to e6ffd7a Compare April 24, 2020 08:58
@AoNoOokami AoNoOokami added this to the release: v0.8.0 milestone Apr 24, 2020
@AoNoOokami AoNoOokami merged commit aad516d into rero:dev Apr 24, 2020
@AoNoOokami AoNoOokami deleted the zaa-#1377-filter-results-by-org branch June 2, 2020 09:22
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.

3 participants