Skip to content

Process embedded share#5569

Merged
glpatcern merged 1 commit intomasterfrom
process-ro-crate
Apr 7, 2026
Merged

Process embedded share#5569
glpatcern merged 1 commit intomasterfrom
process-ro-crate

Conversation

@rawe0
Copy link
Copy Markdown
Member

@rawe0 rawe0 commented Apr 6, 2026

PR to process an embedded share, to be improved once we have the proper CS3APIs requests, since we do not want to use the opaque in the UpdateReceivedOCMShare.

@update-docs
Copy link
Copy Markdown

update-docs bot commented Apr 6, 2026

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@rawe0 rawe0 force-pushed the process-ro-crate branch 3 times, most recently from 9a2259e to 666fa20 Compare April 6, 2026 20:53
@rawe0 rawe0 marked this pull request as ready for review April 7, 2026 07:01
@rawe0 rawe0 force-pushed the process-ro-crate branch from 666fa20 to 78c0ddd Compare April 7, 2026 07:05
Comment thread pkg/share/manager/sql/ocm_shares.go Outdated
@rawe0 rawe0 force-pushed the process-ro-crate branch from 78c0ddd to 40258c6 Compare April 7, 2026 08:25
@rawe0 rawe0 force-pushed the process-ro-crate branch from 40258c6 to c812082 Compare April 7, 2026 09:22
@rawe0 rawe0 requested a review from glpatcern April 7, 2026 09:29
Copy link
Copy Markdown
Member

@glpatcern glpatcern left a comment

Choose a reason for hiding this comment

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

Good to go for now, we'll look into extending the CS3 APIs

@glpatcern glpatcern merged commit 829319c into master Apr 7, 2026
26 checks passed
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