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: subject subdivisions display #828

Merged
merged 1 commit into from
May 9, 2022

Conversation

zannkukai
Copy link
Contributor

Co-authored-by: Renaud Michotte renaud.michotte@gmail.com

Dependencies

rero/rero-ils#2808

How to test?

image

Code review check list

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

@zannkukai zannkukai requested a review from Garfield-fr April 1, 2022 13:16
@zannkukai zannkukai self-assigned this Apr 1, 2022
@github-actions github-actions bot added the f: professional ui Professional interface label Apr 1, 2022
@zannkukai zannkukai added this to the v1.11.0 milestone Apr 1, 2022
@zannkukai zannkukai requested a review from jma April 1, 2022 13:16
@ManaDeweerdt
Copy link
Contributor

ManaDeweerdt commented Apr 21, 2022

  • [not linked??] Public view: wrong display of the document type. Ex: https://ilsdev.test.rero.ch/global/documents/708
    image
    • NPR/05.05: ok!
  • When adding a subdivision, it should be expanded (no need to click on the + sign)
    • NPR/05.05: ok!
      image
  • When adding a new subdivision, it must be placed below the existing subdivisions (not before).
    • NPR/05.05: ok!
  • The subdivisions do not seem to be indexed.
    • NPR/05.05: ok!
  • Subjetcs facet : the subdivisions should create a distinct value in the facet. This is not the case.
    • NPR/05.05: in my opinion, each subdivision should be a distinct facet value, which is not the case. To clarify.
      image
  • The import should be change to add those new specifications.
  • Performance : adding subjects and subdivisions takes quite some time.
    • NPR/05.05: between 6-7 seconds when you add/remove a second value of the same subdivision
  • NPR/05.05: In the tooltip (JSON schema descriptions), the MARC references are not given (there were specified in the issue). Is it on purpose?
  • NPR/05.05: Document detailed view (professional interface), links to corporate bodies (bf:Organisation) are not displayed
    image

@zannkukai
Copy link
Contributor Author

Subjetcs facet : the subdivisions should create a distinct value in the facet. This is not the case.

Is it right ? it seems very strange for me. Take this exemple :

  • You create a subject#S1 for the document#D1. This is a local concept subject without subdivision. The main term is "toto".
  • You create a subject#S2 for the document#D2. This is a local concept subject with subdivisions. The main term is the same than S1 :: "toto". The temporal subdivision is "2022".

If I choose your proposition, both subject will be independent facet values : one on "toto", other on "toto - 2022". If user click on "toto" facet value, then it doesn't retrieve the Document#D2 (but this document has also define the "toto" subject... but with a subdivision).

Additionally, the facet displayed value will be often very long to be displayed on one line.
What's the PO team feeling about this problem ?

@zannkukai
Copy link
Contributor Author

The subdivisions do not seem to be indexed.

Yes, they are (in localhost). But not into subject facet values.

@zannkukai zannkukai force-pushed the zan-fix-issue-2455 branch from c065afc to c91274c Compare May 4, 2022 13:13
Update `ngx-formly` to 5.12.1 to solve `oneOf` jsonSChema selection
problem.

Co-authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai force-pushed the zan-fix-issue-2455 branch from c91274c to 22620d2 Compare May 4, 2022 14:18
@zannkukai zannkukai merged commit 10780c9 into rero:staging May 9, 2022
@zannkukai zannkukai deleted the zan-fix-issue-2455 branch May 9, 2022 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f: professional ui Professional interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants