🎨 [Frontend] UX: Join project with assigned wallet #8324
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
🧪 CI InsightsHere's what we observed from your CI run for 4f74574. 🟢 All jobs passed!But CI Insights is watching 👀 |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR improves the UX for joining projects with assigned wallets by streamlining the process based on user wallet access and informing users about wallet context changes. It also adds safeguards by disabling certain settings when projects are in use.
- Automatically joins users to projects when they have wallet access, skipping the study options dialog
- Provides clear feedback when users cannot join due to wallet access restrictions
- Disables billing settings, service updates, and boot options when projects are being used
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| osparc/study/Utils.js | Refactors state checking methods to reduce code duplication and improve maintainability |
| osparc/desktop/wallets/WalletsList.js | Updates wallet list header text from "Currently in use" to "Preferred" |
| osparc/desktop/wallets/WalletListItem.js | Renames wallet button control from "favourite-button" to "preferred-button" |
| osparc/dashboard/ResourceDetails.js | Adds utility method to disable widgets when projects are in use and applies it to settings tabs |
| osparc/dashboard/ResourceBrowserBase.js | Implements wallet access checking logic and dynamic button text based on project usage state |
services/static-webserver/client/source/class/osparc/dashboard/ResourceBrowserBase.js
Show resolved
Hide resolved
odeimaiz
requested review from
GitHK,
bisgaard-itis,
giancarloromeo,
pcrespov and
sanderegg
giancarloromeo
approved these changes
Sep 8, 2025
|
sanderegg
approved these changes
Sep 8, 2025
16 tasks
This was referenced Sep 18, 2025
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What do these changes do?
This PR improves the UX for joining projects with assigned wallets by streamlining the process based on user wallet access and informing users about wallet context changes. It also adds safeguards by disabling certain settings when projects are in use.
Extra:
New user joins without showing the study options window (user has access to the wallet):
New user can't join because has no access to the wallet):
Inform wallet was changed:
Disable some settings:
Related issue/s
How to test
Dev-ops