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

docs: add isPartOf and seqnum properties to documentation (DEV-216) #145

Merged
merged 2 commits into from Jan 17, 2022

Conversation

irinaschubert
Copy link

resolves DEV-216

@jnussbaum
Copy link
Collaborator

Thanks for your work. Here some questions/feedback:

  1. in rosetta, I was able to make 3 pages that belong to the same book, with the pagenums 2, 2, 3. And I made 1 page belonging to another book, with the pagenum 0. So it is possible to:
    • start with 0
    • skip 1
    • double a pagenum
    • is this behaviour correct?
  2. practical use of seqnum: I guess that the formulation "a client is able to leaf through the parts of a compound object" in the docs is rather theoretical than practical? I don't see a practical use.
  3. delete the following on line 672: "(how to derive a resource or property from another one is not part of this documentation)"
    • rationale: it is described in paragraphs #super and #super_1.
  4. in the following sentence, the word "required" is inaccurate. it is in fact possible to omit the seqnum: line 701: "In case of
    resources that are of type StillImageRepresentation, an additional property derived from seqnum with object IntValue is required."

@irinaschubert
Copy link
Author

irinaschubert commented Jan 17, 2022

Hi Johannes, thanks for the review.

  1. Yes, this is correct. It is (right now) the user's responsibility to set the right pagenums (no double numbers, starting wherever he wants, skipping numbers...). If we want to expand that, it needs to go through product management for priorisation.
  2. The practical use is: you have a book (compound object) with several pages (partOf objects). Defining pages as isPartOf the book, the book can be shown as one object and from there one can leaf through the book. You can try it out with the rosetta project and the Book with Images (book_obj_1) and its ImagePartOfABook parts/pages.
  3. I will adjust that
  4. I don't see the word "required" in the following sentence.

@jnussbaum
Copy link
Collaborator

Thanks for your answers.

  1. That's okay.
  2. That's okay.
  3. OK
  4. It is the very last word of the sentence. The sentence says that the seqnum is required, but in fact, you can omit it.
    • line 701: "In case of resources that are of type StillImageRepresentation, an additional property derived from seqnum with object IntValue is required."

@irinaschubert
Copy link
Author

Ah, now I see/understand. Well, yes, it is not strictly required but without it the compound object is not displayed as such, so using it without seqnum doesn't make sense. So, I leave it like that.

@sonarcloud
Copy link

sonarcloud bot commented Jan 17, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jnussbaum jnussbaum merged commit 09d42a4 into main Jan 17, 2022
@jnussbaum jnussbaum deleted the wip/dev-216-add-part-of-documentation branch January 17, 2022 13:51
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.

None yet

2 participants