-
Notifications
You must be signed in to change notification settings - Fork 32
✨ [Frontend] Functions Browser #8116
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.
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
✨ [Frontend] Functions Browser #8116
Conversation
There was a problem hiding this 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 a Functions Browser feature for the frontend that allows users to view and manage functions created from project templates. The implementation includes both the data layer for function management and the UI components for browsing functions in the dashboard.
- Adds comprehensive Functions store and data model infrastructure
- Implements Functions browser UI with filtering, cards, and details view
- Integrates function support into existing dashboard and workbench components
Reviewed Changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
osparc/store/Functions.js |
New store for function CRUD operations and data management |
osparc/data/model/Function.js |
New data model for function entities |
osparc/dashboard/StudyBrowser.js |
Adds Functions context and browsing capabilities |
osparc/dashboard/ResourceBrowserFilter.js |
Adds Functions filter button to sidebar |
osparc/dashboard/ResourceDetails.js |
Integrates function details view |
osparc/info/FunctionLarge.js |
New function information display component |
osparc/workbench/WorkbenchUI.js |
Refactors constants and adds property initialization check |
osparc/widget/PersistentIframe.js |
Updates function opening logic for new/old versions |
Comments suppressed due to low confidence (1)
services/static-webserver/client/source/class/osparc/dashboard/ResourceBrowserBase.js:696
- [nitpick] The method name changed from 'updateTutorialData' to '_updateTutorialData' but this appears to be inconsistent with other similar methods in the class.
_updateTutorialData: function(tutorialData) {
services/static-webserver/client/source/class/osparc/store/Functions.js
Outdated
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/data/model/Function.js
Outdated
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/info/FunctionLarge.js
Outdated
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/info/FunctionLarge.js
Outdated
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/info/FunctionLarge.js
Outdated
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/info/FunctionLarge.js
Outdated
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/study/StudyPreview.js
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/store/Functions.js
Outdated
Show resolved
Hide resolved
|
@mergify queue |
🟠 Waiting for conditions to match
|
giancarloromeo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👓
pcrespov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀
GitHK
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
wvangeit
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very good. Thanks @odeimaiz
sanderegg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice! you might want a list view as well.
|
@mergify queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at 9b06506 |
|



What do these changes do?
This PR brings the first iteration on the Functions Browser. It includes both the data layer for function management and the UI components for browsing functions in the dashboard.
As discussed, there is a Functions section between the Public Projects and Trash, And the functions coming from the backend will be represented with cards there. When the user clicks on a card, a window is popped up and its metadata is shown in the overview section.
Next steps, once the backend provides the required:
template_id, add also the pipeline view to the pop up windownameanddescriptionRelated issue/s
How to test
Dev-ops