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

rest: make details available for shared workflows #658

Merged

Conversation

DaanRosendal
Copy link
Member

Command Allowed
delete
diff
logs
restart
start
status
stop
share-add
share-remove
share-status
close
open
download
du
ls
mv
prune
rm
upload
retention-rules-list

Closes #651, closes #650

@DaanRosendal DaanRosendal force-pushed the feature/shared-workflow-details branch 2 times, most recently from 7ccdab0 to a78f8b4 Compare December 15, 2023 12:23
@DaanRosendal DaanRosendal force-pushed the feature/shared-workflow-details branch 3 times, most recently from f8aea96 to 6ce6288 Compare March 18, 2024 10:37
@DaanRosendal DaanRosendal force-pushed the feature/shared-workflow-details branch 2 times, most recently from b813ad6 to 7756714 Compare March 20, 2024 13:44
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds a new endpoint to share a workflow with a user.

Closes reanahub/reana-client#680
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
@DaanRosendal DaanRosendal force-pushed the feature/shared-workflow-details branch from 7756714 to d440011 Compare March 20, 2024 13:46
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds a new endpoint to retrieve whom a workflow is shared with.

Closes reanahub/reana-client#686
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds new `shared`, `shared_with` and `shared_by` parameters to the
`get_workflows` endpoint to retrieve workflows along with their sharing
information.

Closes reanahub/reana-client#687
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds two endpoints:
- fetch who shared workflows with the user
- fetch who the user shared workflows with

Closes reanahub#648 and reanahub#649
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds a new endpoint to share a workflow with a user.

Closes reanahub/reana-client#680
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds a new endpoint to retrieve whom a workflow is shared with.

Closes reanahub/reana-client#686
@DaanRosendal DaanRosendal force-pushed the feature/shared-workflow-details branch from d440011 to d1bf9f6 Compare March 20, 2024 14:14
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds new `shared`, `shared_with` and `shared_by` parameters to the
`get_workflows` endpoint to retrieve workflows along with their sharing
information.

Closes reanahub/reana-client#687
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds two endpoints:
- fetch who shared workflows with the user
- fetch who the user shared workflows with

Closes reanahub#648 and reanahub#649
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds a new endpoint to share a workflow with a user.

Closes reanahub/reana-client#680
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
DaanRosendal added a commit to DaanRosendal/reana-server that referenced this pull request Mar 20, 2024
Adds a new endpoint to retrieve whom a workflow is shared with.

Closes reanahub/reana-client#686
@mdonadoni
Copy link
Member

Latest version before rebase: ef5010e

There are still some comments left to be addressed from previous PRs.

mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
Adds a new endpoint to share a workflow with a user.

Closes reanahub/reana-client#680
mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
@mdonadoni mdonadoni force-pushed the feature/shared-workflow-details branch from 8c9ecaf to f66d666 Compare August 29, 2024 10:08
mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
Adds a new endpoint to retrieve whom a workflow is shared with.

Closes reanahub/reana-client#686
mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
Adds new `shared`, `shared_with` and `shared_by` parameters to the
`get_workflows` endpoint to retrieve workflows along with their sharing
information.

Closes reanahub/reana-client#687
mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
Adds two endpoints:
- fetch who shared workflows with the user
- fetch who the user shared workflows with

Closes reanahub#648 and reanahub#649
mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
mdonadoni pushed a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
mdonadoni added a commit to DaanRosendal/reana-server that referenced this pull request Aug 29, 2024
@mdonadoni
Copy link
Member

Ready to be reviewed, squashed and merged

@tiborsimko tiborsimko force-pushed the feature/shared-workflow-details branch from f66d666 to 50495fc Compare September 4, 2024 16:47
Copy link
Member

@tiborsimko tiborsimko left a comment

Choose a reason for hiding this comment

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

Works nicely 👍 with

- r-client #692
- r-client-go #153
- r-ui #375
- r-server #658
- r-workflow-controller #552
- r-commons #429
- r-db #216

Kept separate commits as elsewhere, amended commit log headlines, increased reana-db and pytest-reana dependency versions.

Copy link

codecov bot commented Sep 4, 2024

Codecov Report

Attention: Patch coverage is 42.69663% with 51 lines in your changes missing coverage. Please review.

Project coverage is 60.01%. Comparing base (d1cd1a5) to head (50495fc).
Report is 9 commits behind head on master.

Files with missing lines Patch % Lines
reana_server/rest/workflows.py 35.29% 33 Missing ⚠️
reana_server/rest/users.py 52.63% 18 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #658      +/-   ##
==========================================
- Coverage   60.84%   60.01%   -0.83%     
==========================================
  Files          33       32       -1     
  Lines        3463     3509      +46     
==========================================
- Hits         2107     2106       -1     
- Misses       1356     1403      +47     
Files with missing lines Coverage Δ
reana_server/rest/users.py 42.22% <52.63%> (+5.18%) ⬆️
reana_server/rest/workflows.py 51.04% <35.29%> (-2.46%) ⬇️

... and 31 files with indirect coverage changes

@tiborsimko tiborsimko merged commit 50495fc into reanahub:master Sep 4, 2024
14 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
3 participants