Skip to content

🎨 [Frontend] Hide Sim4Life->oSparc integration features #7207

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 13 commits into from
Feb 12, 2025
Merged

🎨 [Frontend] Hide Sim4Life->oSparc integration features #7207

merged 13 commits into from
Feb 12, 2025

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Feb 11, 2025 •
edited
Loading

What do these changes do?

As conclusions from 10.02 meeting:

  • Remove pipelining (oSparc), iseg and also the sections from Sim4Life's plus button's menu (leave new folder and sim4life and jlab only)
  • Make Plus button's menu more distinguishable (strong-color border)
  • Bring back Tutorial and Services tab buttons
  • Hide Convert to Pipeline options
  • Re-rename "Bin" to "Recently Deleted" and "Move to Bin" to "Delete" and "Moved to bin" to "Deleted"
  • Replace the Download logs button in Study Editor's menu by the Platform Logger

HideSomeFeatures

Related issue/s

How to test

Dev-ops checklist

@odeimaiz odeimaiz self-assigned this Feb 11, 2025
@odeimaiz odeimaiz added this to the Singularity milestone Feb 11, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) labels Feb 11, 2025
@odeimaiz odeimaiz changed the title 🎨 [Frontend] Hide Sim4Life oSparc integration 🎨 [Frontend] Hide Sim4Life->oSparc integration features Feb 11, 2025
@odeimaiz odeimaiz marked this pull request as ready for review February 11, 2025 14:32
@odeimaiz odeimaiz enabled auto-merge (squash) February 11, 2025 14:39
GitHK
GitHK approved these changes Feb 11, 2025
View reviewed changes
Copy link
Contributor

@GitHK GitHK 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

pcrespov
pcrespov reviewed Feb 11, 2025
View reviewed changes
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.

👍 +1:

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 12, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@odeimaiz odeimaiz merged commit c04a389 into ITISFoundation:master Feb 12, 2025
57 checks passed
@odeimaiz odeimaiz deleted the request/hide-osparc-integration branch February 12, 2025 10:56
@calys
Copy link

calys commented Feb 13, 2025

👍

@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Mar 6, 2025
Closed
63 tasks
mrnicegyu11 pushed a commit to mrnicegyu11/osparc-simcore that referenced this pull request Mar 26, 2025
@odeimaiz @mrnicegyu11
🎨 [Frontend] Hide Sim4Life->oSparc integration features (ITISFoundati…
81d9232 
…on#7207)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pcrespov pcrespov pcrespov left review comments

@ignapas ignapas ignapas approved these changes

@GitHK GitHK GitHK approved these changes

@matusdrobuliak66 matusdrobuliak66 matusdrobuliak66 approved these changes

@jsaq007 jsaq007 Awaiting requested review from jsaq007

@giancarloromeo giancarloromeo Awaiting requested review from giancarloromeo

@calys calys Awaiting requested review from calys

@mguidon mguidon Awaiting requested review from mguidon

@sanderegg sanderegg Awaiting requested review from sanderegg

Assignees

@odeimaiz odeimaiz

Labels

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

Projects

None yet

Milestone

Singularity

Development

Successfully merging this pull request may close these issues.

6 participants

@odeimaiz @calys @ignapas @GitHK @pcrespov @matusdrobuliak66