Skip to content

Conversation

@wvangeit
Copy link
Contributor

@wvangeit wvangeit commented May 20, 2025

What do these changes do?

There was a small inconsistency between the frontend function definition and the backend, introduced during changes in the functions API PR. It prevent the input/output schema's ending up in the db.
Also a small change in how default inputs are handled. Default inputs are only used when the parameter is not exposed by the user.
Also the function name is the 'title' field (in sync with other osparc objects).

How to test

Create a function in the frontend. Check in postgres if funcapi_functions table has the correct function with schema.

Dev-ops

No change ENV

@wvangeit wvangeit added this to the Bazinga! milestone May 20, 2025
@wvangeit wvangeit self-assigned this May 20, 2025
@wvangeit wvangeit requested a review from odeimaiz as a code owner May 20, 2025 13:51
@wvangeit wvangeit added the a:frontend issue affecting the front-end (area group) label May 20, 2025
@sonarqubecloud
Copy link

@wvangeit
Copy link
Contributor Author

@mergify queue

@mergify
Copy link
Contributor

mergify bot commented May 20, 2025

queue

βœ… The pull request has been merged automatically

The pull request has been merged automatically at 5c86393

@wvangeit wvangeit added the πŸ€–-automerge marks PR as ready to be merged for Mergify label May 20, 2025
@sanderegg sanderegg self-requested a review May 20, 2025 14:29
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.

πŸ”Ή

@mergify mergify bot merged commit 5c86393 into ITISFoundation:master May 20, 2025
59 checks passed
@wvangeit wvangeit deleted the fix_functions_frontend_schema branch May 20, 2025 14:35
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Jun 6, 2025
92 tasks
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Aug 5, 2025
88 tasks
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)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants