Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Oct 10, 2025

What do these changes do?

This PR adds a new "Transfer Projects" feature that allows users to share all the projects they own (delete access) with another user.

There are two ways to transfer all your projects to another user:

  • Share all your projects with the target user and keep the co-ownership
  • Share all your projects with the target user and remove yourself as co-owner

For this first iteration, to be on the safe side, only the first way will be enabled.

Transfer projects:
TransferProjects

Target user:
TransferProjects_2

Bonus:

  • let testers change the template type
  • show "Only users with write access can add comments." disclaimer if one can't participate in a conversation

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Oct 10, 2025
@odeimaiz odeimaiz added this to the Cheops milestone Oct 10, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) labels Oct 10, 2025
@mergify
Copy link
Contributor

mergify bot commented Oct 10, 2025

🧪 CI Insights

Here's what we observed from your CI run for 3bb9b89.

🟢 All jobs passed!

But CI Insights is watching 👀

@odeimaiz odeimaiz requested a review from Copilot October 10, 2025 14:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new "Transfer Projects" feature that allows users to share all their projects with another user while optionally keeping co-ownership. It also includes two smaller improvements: allowing testers (not just product owners) to change template types, and displaying a disclaimer when users lack write access to add comments in conversations.

  • Added transfer projects functionality with UI for selecting target users and sharing options
  • Updated permission checks to allow testers to modify template types
  • Enhanced conversation UI to show permission disclaimers for read-only users

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
osparc/store/Study.js Added getAllMyStudies method to fetch all user studies for transfer operations
osparc/info/StudyLarge.js Changed permission check from product owner to tester for template type modifications
osparc/desktop/account/TransferProjects.js New component implementing the transfer projects interface and logic
osparc/desktop/account/ProfilePage.js Integrated transfer projects section into the user profile page
osparc/conversation/AddMessage.js Added footer layout and permission disclaimer for users without write access

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sonarqubecloud
Copy link

@odeimaiz odeimaiz requested a review from capbrown October 10, 2025 15:30
@odeimaiz odeimaiz marked this pull request as ready for review October 10, 2025 15:30
@matusdrobuliak66
Copy link
Collaborator

matusdrobuliak66 commented Oct 10, 2025

Q: When its shared and removed or only shared --> What kind of privilages are given to the user? the new user becomes a "owner"?

Copy link
Collaborator

@matusdrobuliak66 matusdrobuliak66 left a comment

Choose a reason for hiding this comment

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

cool

@odeimaiz
Copy link
Member Author

odeimaiz commented Oct 10, 2025

Q: When its shared and removed or only shared --> What kind of privileges are given to the user? the new user becomes a "owner"?

Yep, the target user becomes a co-owner, full permission.

@odeimaiz odeimaiz enabled auto-merge (squash) October 10, 2025 15:51
@odeimaiz odeimaiz merged commit b01bc0d into ITISFoundation:master Oct 10, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants