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

documents: display 'part of' in brief views #506

Merged

Conversation

AoNoOokami
Copy link
Contributor

@AoNoOokami AoNoOokami commented Feb 2, 2021

Co-Authored-by: Alicia Zangger [email protected]

Why are you opening this PR?

To solve the issue mentioned above.

Dependencies

My PR depends on rero-ils#<xx>'s PR(s):

N/A

How to test?

See issue description.

Code review check list

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

@AoNoOokami AoNoOokami self-assigned this Feb 2, 2021
@AoNoOokami AoNoOokami added f: professional ui Professional interface f: public ui Public interface, as opposed to the professional interface labels Feb 2, 2021
@AoNoOokami AoNoOokami force-pushed the zaa-#1596-display-partof-brief-views branch 2 times, most recently from 430fb87 to 24baeff Compare February 3, 2021 10:07
@AoNoOokami AoNoOokami marked this pull request as ready for review February 3, 2021 10:08
@AoNoOokami AoNoOokami force-pushed the zaa-#1596-display-partof-brief-views branch 2 times, most recently from 83ab3c4 to 92bbb2c Compare February 3, 2021 10:16
@AoNoOokami AoNoOokami modified the milestone: v1.1.0 Feb 3, 2021
@iGormilhit iGormilhit added this to the v1.1.0 milestone Feb 4, 2021
Copy link
Contributor

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

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

In the document title, set documents to the plural form.

@AoNoOokami AoNoOokami force-pushed the zaa-#1596-display-partof-brief-views branch from 92bbb2c to 0d55509 Compare February 8, 2021 13:21
@AoNoOokami AoNoOokami changed the base branch from dev to v_1_1_0 February 8, 2021 15:02
@AoNoOokami AoNoOokami changed the base branch from v_1_1_0 to dev February 9, 2021 10:35
@iGormilhit iGormilhit changed the title document: display 'part of' in brief views documents: display 'part of' in brief views Feb 10, 2021
@iGormilhit iGormilhit self-requested a review February 10, 2021 06:41
@AoNoOokami AoNoOokami force-pushed the zaa-#1596-display-partof-brief-views branch from 0d55509 to 40c2f5f Compare February 17, 2021 15:42
@AoNoOokami AoNoOokami requested a review from zannkukai February 17, 2021 15:42
* Moves 'part of' template and code to a new component in shared
library. This component is used in detailed and brief views.
* Closes rero/rero-ils#1596.

Co-Authored-by: Alicia Zangger <[email protected]>
@AoNoOokami AoNoOokami force-pushed the zaa-#1596-display-partof-brief-views branch from 40c2f5f to 2cb5959 Compare February 22, 2021 07:06
@AoNoOokami AoNoOokami requested a review from zannkukai February 22, 2021 10:08
@AoNoOokami AoNoOokami merged commit 17f5813 into rero:dev Feb 22, 2021
@iGormilhit iGormilhit modified the milestones: v1.1.0, v1.0.1 Mar 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f: professional ui Professional interface f: public ui Public interface, as opposed to the professional interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Display partOf on the brief views
4 participants