Skip to content

Conversation

@wvangeit
Copy link
Contributor

@wvangeit wvangeit commented May 26, 2025

What do these changes do?

Mark inputs as required in the function schema in the create function frontend

How to test

Create a function in frontend, test if function schema has 'required' field with the expected variables, by calling api/checking db.

Dev-ops

no changes

@wvangeit wvangeit self-assigned this May 26, 2025
@wvangeit wvangeit added the a:frontend issue affecting the front-end (area group) label May 26, 2025
@wvangeit wvangeit added this to the Bazinga! milestone May 26, 2025
@wvangeit wvangeit marked this pull request as ready for review May 26, 2025 12:59
@wvangeit wvangeit requested a review from odeimaiz as a code owner May 26, 2025 12:59
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.

👀

@wvangeit
Copy link
Contributor Author

wvangeit commented Jun 2, 2025

@mergify queue

@wvangeit wvangeit added the 🤖-automerge marks PR as ready to be merged for Mergify label Jun 2, 2025
@mergify
Copy link
Contributor

mergify bot commented Jun 2, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at cc3dbbe

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 2, 2025

@mergify mergify bot merged commit cc3dbbe into ITISFoundation:master Jun 2, 2025
59 checks passed
@wvangeit wvangeit deleted the add_required_in_createfunction branch June 2, 2025 07:33
@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