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

Group/share results #948

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from
Draft

Group/share results #948

wants to merge 14 commits into from

Conversation

malchua
Copy link
Contributor

@malchua malchua commented Feb 14, 2025

What does this PR do and why?

Describe in detail what your merge request does and why.
This PR relates to STRY0016470, where we want to be able to share workflow executions to the group so that members can see the results. This PR covers issue #885

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other pull requests.
image

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Login as a user that is a member of a group.
  2. Go to that groups page.
  3. In the samples page for that group, create a workflow execution and check the box asking if you want to share the results with the group members.
  4. After creating the workflow execution, click on the Workflow Executions section on the sidebar of that group.
  5. Confirm that the workflow execution is present.
  6. Log out, then login as a different member that has access to the same group (the access level must be at least an Analyst)
  7. Go to the Workflow Executions page for the same group.
  8. Confirm that this member is also able to see the same workflow execution.
  9. Confirm that this member is not allowed to Cancel, Delete, or Edit this workflow execution.
  10. Select the workflow execution and create an export from it.
  11. Go to the user's data export listing and see that the data export has been created and all the links work properly.

PR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

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.

1 participant