sim icon indicating copy to clipboard operation
sim copied to clipboard

feat(better-error-handling): Added a method to check al the required sub-blocks of the workflow blocks.

Open mo-amir-code opened this issue 8 months ago • 7 comments

Description

Added a method to validate all the required sub-block values of each block of current workflow.

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context:

I added a method to check all the required sub-blocks values of workflow blocks. If any required field is missing then it is showing error alert/notification message only for first detected unfilled required sub-block value. For example: user created a workflow with 4 blocks and missed 2 required sub-block values in two different block then that method will show alert message for first unfilled sub-block value because it is checking flow wise and if user filled that required sub-block value and then the method will give an error message again to another unfilled sub-block value.

Fixes #340

Type of change

Please delete options that are not relevant.

  • [x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] All tests pass locally and in CI (bun run test)
  • [x] My changes generate no new warnings
  • [ ] Any dependent changes have been merged and published in downstream modules
  • [ ] I have updated version numbers as needed (if needed)
  • [x] I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

Security Considerations:

  • [x] My changes do not introduce any new security vulnerabilities
  • [ ] I have considered the security implications of my changes

Additional Information:

Any additional information, configuration or data that might be necessary to reproduce the issue or use the feature.

mo-amir-code avatar May 18 '25 11:05 mo-amir-code

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 18, 2025 1:08pm

vercel[bot] avatar May 18 '25 11:05 vercel[bot]

@mo-amir-code is attempting to deploy a commit to the Sim Studio Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar May 18 '25 11:05 vercel[bot]

@mo-amir-code the use workflow execution doesn't only run in the UI, but on the API, webhook, and chat message. How can we return a value so that an error is displayed for those?

emir-karabeg avatar May 18 '25 23:05 emir-karabeg

@emir-karabeg I was thinking only in UI, I will start working on it

mo-amir-code avatar May 19 '25 00:05 mo-amir-code

@mo-amir-code any updates? :D

emir-karabeg avatar May 26 '25 08:05 emir-karabeg

@emir-karabeg I wanted to let you know that I’m currently unable to continue working on the PR due to my ongoing college exams. As the exams are still in progress, I won’t be able to contribute further at the moment.

Please feel free to reassign or let someone else take over the task if needed. I really appreciate the opportunity to contribute and hope to be more active once my exams are over.

Thanks for your understanding.

mo-amir-code avatar May 27 '25 01:05 mo-amir-code

@mo-amir-code no problem! Best of luck with your exams and looking forward to hearing from you soon! Let us know if we can help in any way :D @waleedlatif1 and I studied CS at Berkeley

emir-karabeg avatar May 27 '25 02:05 emir-karabeg

resolved by #843

waleedlatif1 avatar Aug 06 '25 02:08 waleedlatif1