Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Sep 9, 2025

What do these changes do?

This PR implements visual distinction between product groups (Everyone/Public and Support) and regular groups in the frontend interface. The changes add visual separators and reorder collaborators to prioritize product-related groups.

Bonus:

ProductGroups

Related issue/s

How to test

Dev-ops

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

mergify bot commented Sep 9, 2025

🧪 CI Insights

Here's what we observed from your CI run for 7469de8.

✅ Passed Jobs With Interesting Signals

Pipeline Job Signal Health on master Retries 🔍 CI Insights 📄 Logs
CI system-tests Base branch is broken, but the job passed. Looks like this might be a real fix 💪 Broken 0 View View
unit-tests Base branch is broken, but the job passed. Looks like this might be a real fix 💪 Broken 0 View View

@odeimaiz odeimaiz marked this pull request as ready for review September 9, 2025 11:59
@odeimaiz odeimaiz requested a review from Copilot September 9, 2025 11:59
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 implements visual distinction between product groups (Everyone/Public and Support) and regular groups in the frontend interface. The changes add visual separators and reorder collaborators to prioritize product-related groups.

Key changes:

  • Introduces a standardized ordering system for collaboration types
  • Adds visual separators between product groups and other groups in various UI components
  • Updates icons for better visual distinction of support groups

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
osparc/theme/Decoration.js Adds "separator-strong" decoration style for visual separation
osparc/store/Groups.js Defines collaboration type ordering array for consistent sorting
osparc/share/Collaborators.js Implements product group sorting and separator insertion logic
osparc/filter/OrganizationsAndMembers.js Updates to use centralized collaboration type ordering
osparc/filter/CollaboratorToggleButton.js Adds icon width standardization for alignment
osparc/desktop/organizations/OrganizationsList.js Adds separator logic and safe property access
osparc/data/model/Group.js Fixes syntax error in statics object
osparc/dashboard/CardBase.js Updates support group icon

@odeimaiz odeimaiz added the 🤖-automerge marks PR as ready to be merged for Mergify label Sep 9, 2025
@odeimaiz
Copy link
Member Author

odeimaiz commented Sep 9, 2025

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented Sep 9, 2025

queue

🛑 Configuration not compatible with a branch protection setting

The branch protection setting Require branches to be up to date before merging is not compatible with max_parallel_checks>1, queue_conditions != merge_conditions and must be unset.

Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

thx

I would make the separator a bit bigger

Copy link
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

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

very nice addition!

@odeimaiz odeimaiz enabled auto-merge (squash) September 9, 2025 14:59
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 9, 2025

@odeimaiz odeimaiz merged commit 7f0830f into ITISFoundation:master Sep 9, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖-automerge marks PR as ready to be merged for Mergify 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.

[Frontend] Support Info Not Displayed for /me Endpoint Errors with supportID

3 participants