Skip to content

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

Merged
jnussbaum merged 2 commits into
mainfrom
wip/dev-216-add-part-of-documentation
Jan 17, 2022
Merged

jnussbaum merged 2 commits into
mainfrom
wip/dev-216-add-part-of-documentation

Conversation

@irinaschubert

Copy link
Copy Markdown

resolves DEV-216

@irinaschubert irinaschubert self-assigned this Jan 14, 2022
@jnussbaum

Copy link
Copy Markdown
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

irinaschubert commented Jan 17, 2022 •

Copy link
Copy Markdown
Author

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
Copy Markdown
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
Copy Markdown
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.

@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants