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

Data transfers new ocm impl #3847

Merged
merged 3 commits into from
May 4, 2023
Merged

Conversation

redblom
Copy link
Contributor

@redblom redblom commented May 4, 2023

Concerns #3846
. remove unnecessary creation of ocm reference in ocmshareprovider when accepting an ocm share
. set transfer protocol when creating transfer type ocm share
. refactor transfer endpoints for new ocm impl
. refactor/cleanup gateway.ocmshareprovider.UpdateReceivedOCMShare() code
. refactor data transfers folder config
. new transfers config setting 'remove_on_cancel'
. implement transfer destination path
. update datatx example toml
. update cli ocm-share-update-received with path flag

@redblom redblom requested review from labkode, gmgigi96, glpatcern and a team as code owners May 4, 2023 10:03
Copy link
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.

Nice work! It would be even nicer to not have opaque fields, but I think we can go ahead with this.

@redblom
Copy link
Contributor Author

redblom commented May 4, 2023

Nice work! It would be even nicer to not have opaque fields, but I think we can go ahead with this.

Yes, let's save that for the next CS3APIs upgrade.

…en accepting an ocm share

* set transfer protocol when creating transfer type ocm share
* refactor transfer endpoints for new ocm impl
* refactor/cleanup gateway.ocmshareprovider.UpdateReceivedOCMShare() code
* refactor data transfers folder config
* new transfers config setting 'remove_on_cancel'
* implement transfer destination path
* update datatx example toml
* update cli ocm-share-update-received with path flag
@redblom redblom force-pushed the datatx-new-ocm-impl branch 6 times, most recently from 9d18aa4 to ed861f8 Compare May 4, 2023 15:10
@glpatcern glpatcern merged commit 80606f0 into cs3org:master May 4, 2023
gmgigi96 pushed a commit to gmgigi96/reva that referenced this pull request Jun 5, 2023
* * remove unnecessary creation of ocm reference in ocmshareprovider when accepting an ocm share
* set transfer protocol when creating transfer type ocm share
* refactor transfer endpoints for new ocm impl
* refactor/cleanup gateway.ocmshareprovider.UpdateReceivedOCMShare() code
* refactor data transfers folder config
* new transfers config setting 'remove_on_cancel'
* implement transfer destination path
* update datatx example toml
* update cli ocm-share-update-received with path flag

* Add changelog

* Add #PR

---------

Co-authored-by: Antoon P <antoon@redblom.com>
gmgigi96 pushed a commit to gmgigi96/reva that referenced this pull request Jun 28, 2023
* * remove unnecessary creation of ocm reference in ocmshareprovider when accepting an ocm share
* set transfer protocol when creating transfer type ocm share
* refactor transfer endpoints for new ocm impl
* refactor/cleanup gateway.ocmshareprovider.UpdateReceivedOCMShare() code
* refactor data transfers folder config
* new transfers config setting 'remove_on_cancel'
* implement transfer destination path
* update datatx example toml
* update cli ocm-share-update-received with path flag

* Add changelog

* Add #PR

---------

Co-authored-by: Antoon P <antoon@redblom.com>
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.

3 participants