Conversation
|
@kishanprmr @onyedikachi-david ready for a review in this one. |
| description: | ||
| 'The question to ask the bot. 2 to 500 characters. Max increased to 8K tokens (roughtly 32k chars) when authenticated.', | ||
| }), | ||
| conversationId: Property.ShortText({ |
sanket-a11y
left a comment
There was a problem hiding this comment.
please check requested changes
|
Hey @sanket-a11y, thank you for your review. I left some comments with questions. I'm currently validating the conditional required fields with a Zod Schema, as I pointed out in my comments, do you think that's enough? If not, where do you suggest me to make those checks? |
|
@sanket-a11y, implemented all other changes, only my question about the conditional validation to go. |
|
@LuizDMM For the validation you can use the DynamicProperties prop |
Got it, on it. |
|
@sanket-a11y Fixed the |
|
Just a demo with the DocsBot2.mp4 |
|
View your CI Pipeline Execution ↗ for commit 8a57d63
☁️ Nx Cloud last updated this comment at |
What does this PR do?
This PR delivers the full implementation of the DocsBot integration, completing all the required actions with strong validation and type safety.
Completed Scope
How the Feature Works
All actions were fully implemented and carefully validated to prevent invalid requests. Every function is strongly typed according to the API documentation, ensuring reliability and maintainability.
The Create Source action includes a custom Zod schema that handles conditionally required fields, adding an extra layer of robustness to the validation process.
Additionally, here's a demo video showcasing the feature in action:
DocsBot.mp4
Fixes #9049
/claim #9049
Edit: Added demo video.