-
Notifications
You must be signed in to change notification settings - Fork 3.3k
feat(database): tools added #211
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
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@RadoBoiii is attempting to deploy a commit to the Sim Studio Team on Vercel. A member of the Team first needs to authorize it. |
|
@RadoBoiii nice. did we test these tools out? Also, the elasticsearch and redis are missing from the tools/index.ts, I'm not sure if that's intentional or not. |
waleedlatif1
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.
let's remove packages/sdk from the root dir. They may have appeared if you ran npm run dev in the root of the repo, which generates a package.json at the root. Also, let's revert the changes we made to gitignore
I'm yet to implement them, so I haven't added their tool files yet. Are the other tools running as expected? I will do it once I get everything to work. I will remove the packages/sdk folders and revert the gitignore files |
- included index.ts for redis nd elasticsearch. - reverted gitignore to original - removed packages/sdk from root
restored packages/sdk
- tested mysql and postgre sql
made changes
updated packages/sdk
b3ac421 to
edd5612
Compare
…in token refresh for airtable
…ol bar refreshing
* feat(autofill): autofill api keys for tools once they are set once * added setting to settings for envvar autofill, removed from agent block since it depends on the model selected * add in api key autofill for agent blocks based on existing model provider mapping * add tooltips for general settings * resolved pr comments * reorganized files, removed unused imports
…ified store that handles sub-block values (#237)
* fix(loops): added resolve for string collections and added single block loop * improvement(loop-input): error validation removed * improvement(loops): fixed now for source/external blocks; doesn't work for multi-block loop (>2) * fix(loops): complex looping functions fully work * improvement: cleaned up debugging logs
- included index.ts for redis nd elasticsearch. - reverted gitignore to original - removed packages/sdk from root
restored packages/sdk
- tested mysql and postgre sql
made changes
|
creating a new PR |
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
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:
npm test)Security Considerations:
Additional Information:
Any additional information, configuration or data that might be necessary to reproduce the issue or use the feature.