🚀 SID personal data integration #1244
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A new import 'SIDSearch' from 'reworkd_platform.web.api.agent.tools' has been added to the 'tools.py' file. Now, the SIDSearch tool is also available for use along with Image, Search, and Code.
…ity (add-sid) Add a blank line in search.py. Change database settings in settings.py and add refresh token for debug. Create new file sidsearch.py that implements SID search functionality through various platforms like Google Drive, Notion, GMail, etc. using user's personal refresh token.
Added a comment in sidsearch.py to clarify that the access token should not be hardcoded, but obtained through an auth token exchange. The Bearer token in the 'Authorization' header has been changed as well.
…s and functionality (add-sid) The new ConnectSidButton component has been created and added to the settings page. This component provides functionality for connecting and disconnecting SID, with different button styles for different states. The button dimensions and font scale can be customized via component props. Disconnect functionality is currently a placeholder and needs to be implemented. In addition, minor formatting adjustments were made to the settings page for better readability and consistency.
…-sid) The authorization field in the headers of the _sid_search_results function has been updated to a placeholder. Additionally, the description field of the SIDSearch class has been revised to provide more accurate information about the tool's functionality.
|
@gvilums is attempting to deploy a commit to the reworkd Team on Vercel. A member of the Team first needs to authorize it. |
|
Hi 👋 Here's a preview environment 🚀 https://next-reworkd-agentgpt-1244.env.ergomake.link Environment Summary 📑
Here are your environment's logs. For questions or comments, join Discord. Click here to disable Ergomake. |
awtkns
reviewed
Aug 20, 2023
Contributor
awtkns
reviewed
Aug 20, 2023
awtkns
reviewed
Aug 20, 2023
awtkns
reviewed
Aug 20, 2023
| return False | ||
|
|
||
| # use refresh token to revoke access | ||
| delete_token = encryption_service.decrypt(creds.refresh_token_enc) |
Contributor
There was a problem hiding this comment.
Might be best just to remove the row from the db here. How would i re-authenticate?
Contributor
Author
There was a problem hiding this comment.
When implementing I though that only soft deletes were supported. I changed it to a hard delete, such that the row is actually removed from the db.
awtkns
reviewed
Aug 20, 2023
awtkns
reviewed
Aug 20, 2023
awtkns
suggested changes
Aug 20, 2023
platform/reworkd_platform/web/api/agent/agent_service/open_ai_agent_service.py
Outdated
Show resolved
Hide resolved
Contributor
|
@awtkns I am still cleaning some stuff up right now |
Contributor
Author
|
@awtkns We finished the integration of the button and applied the suggestions you made. This should be ready for final review now. |
awtkns
reviewed
Aug 21, 2023
awtkns
approved these changes
Aug 21, 2023
Contributor
|
Looks good to me. Have some changes I will push up in the morning to fixup styling, typing and some minor refactors. We can go live tmrw! |
# Conflicts: # platform/reworkd_platform/web/api/auth/views.py
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces SID as a new tool available to the agent. The changes mostly concern