Skip to content

Allow users to cascade the sharing - #2038

Merged
aglinxinyuan merged 10 commits into
masterfrom
xinyuan-change-sharing
Jul 7, 2023
Merged

Allow users to cascade the sharing#2038
aglinxinyuan merged 10 commits into
masterfrom
xinyuan-change-sharing

Conversation

@aglinxinyuan

@aglinxinyuan aglinxinyuan commented Jul 6, 2023

Copy link
Copy Markdown
Contributor

In this PR

  1. Allows users to cascade the sharing.
    For example, User A shared a workflow with user B, and user B can share the same workflow with User C.
    The same logic is applied to workflow, files, and projects. Resolve issue Cascade resource sharing #2035

  2. Some mirror refactoring.

  3. Fix the UI bug because the top bar blocks the first workflow's owner information.

image
  1. People can revoke the resources shared with them.

@aglinxinyuan aglinxinyuan self-assigned this Jul 7, 2023

@Yicong-Huang Yicong-Huang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Left minor comments.

Comment thread core/new-gui/src/app/dashboard/user/type/dashboard-project.interface.ts Outdated
@aglinxinyuan
aglinxinyuan merged commit e35b8d8 into master Jul 7, 2023
@aglinxinyuan
aglinxinyuan deleted the xinyuan-change-sharing branch July 7, 2023 09:22
yangzhang75 pushed a commit to yangzhang75/texera that referenced this pull request Jun 22, 2026
In this PR

1. Allows users to cascade the sharing.
For example, User **A** shared a workflow with user **B**, and user
**B** can share the same workflow with User **C**.
The same logic is applied to workflow, files, and projects. Resolve
issue apache#2035

2. Some mirror refactoring.

3. Fix the UI bug because the top bar blocks the first workflow's owner
information.
<img width="502" alt="image"
src="https://github.com/Texera/texera/assets/11544314/2f325902-a81c-4d82-9cc5-431ef31c1856">

4. People can revoke the resources shared with them.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants