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

Simplify references for retrieval #130

Merged
merged 11 commits into from
Jun 14, 2021

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Jun 7, 2021

cleaned up version of #126 as a new PR to have them comparable.

  • simplifies the references into a combined Reference with storage_id, node_id and path
  • leaves property names for id based references as resource_id but does change the type. The share implementations need to use ids to prevent breaking references when a path changes.
  • leaves the type string Path property in ResourceInfo

@butonic butonic requested a review from labkode as a code owner June 7, 2021 15:00
@butonic butonic force-pushed the simplify-references-for-retrieval branch 3 times, most recently from 02b9c2d to 5133cda Compare June 11, 2021 15:08
@C0rby C0rby force-pushed the simplify-references-for-retrieval branch 2 times, most recently from 3d99efb to fef7e6c Compare June 11, 2021 15:50
butonic and others added 11 commits June 11, 2021 17:52
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@C0rby C0rby force-pushed the simplify-references-for-retrieval branch from fef7e6c to fd1c41d Compare June 11, 2021 15:53
@butonic
Copy link
Contributor Author

butonic commented Jun 14, 2021

@ishank011 all updated

@ishank011 ishank011 merged commit 1a4ab6b into cs3org:main Jun 14, 2021
glpatcern pushed a commit to cs3org/wopiserver that referenced this pull request Jun 24, 2021
glpatcern pushed a commit to cs3org/wopiserver that referenced this pull request Jun 24, 2021
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.

4 participants