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.

Sign up for GitHub

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

Jump to bottom

refactor: make owner_email for owned workflows the email of the owner #662

Closed
DaanRosendal opened this issue Dec 15, 2023 · 1 comment
Closed

refactor: make owner_email for owned workflows the email of the owner #662

DaanRosendal opened this issue Dec 15, 2023 · 1 comment
Labels
system/refactor type/enhancement

Comments

@DaanRosendal
Copy link
Member

Currently, owner_email is set to "-" for owned workflows. This plays nicely with the CLI, but makes code less readable in most places.

Beware of the side-effects of this change, e.g. the CLI should still output a "-" for owned workflows when the user exectutes the list --shared command.
@mdonadoni
Copy link
Member

Done as part of reanahub/reana-workflow-controller@e36c6a2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
system/refactor type/enhancement
Projects
Workflow-Sharing
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mdonadoni @DaanRosendal