chore: add conditionals for supabase to agent directory #2032
Conversation
This file contains 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
Added Condional for Supabase to the initializeDatabase function to allow for ease of use for Supabase as well as keeping the postgres and sqlite funcitonality
Merged
odilitime
previously approved these changes
Jan 8, 2025
odilitime
changed the title
|
I also would like to update the documentation that is at https://eliza.gg/eliza/docs/packages/adapters/ under Schema Management, PostgreSQL Schema. The Sql is incorrect and causes errors. I have the correct sql that ran on Supabase without error. When i look at the docs on the main branch of this repo. The lines that need edited do not exist. Where can i make this change? |
Coreected the supabase import to match the proper format.
Added adapter-supabase to packag.json so that the adapter will not fail to load.
0xpi-ai
pushed a commit
to 0xpi-ai/NayariAI
that referenced
this pull request
Jan 15, 2025
chore: add conditionals for supabase to agent directory
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Added Condional for Supabase to the initializeDatabase function to allow for ease of use for Supabase as well as keeping the postgres and sqlite functionality. In conjunction with #2031
Relates to
N/A
Risks
Low
Background
As it stands, beginner users must add the Supabase functionality manually to use Supabase.
What does this PR do?
his PR allows for them to simply add their SUPABASE_URL and SUPABASE_ANON_KEY to the .env to use supabase as their db.
What kind of change is this?
Improvements (misc. changes to existing features)
Why are we doing this? Any context or related work?
As a user it is very difficult to use the Supabase adapter. This makes it much easier for a beginner user to not have to code any new functionality to use the adapter. In conjunction with #203 the user can simply add their SUPABASE_URL and SUPABASE_ANON_KEY to the .env to use supabase as their db.
Documentation changes needed?
My changes do not require a change to the project documentation, but if needed and you tell me where to update it, I can .
Testing
Needs testing.
Where should a reviewer start?
Detailed testing steps
As user add SUPABASE_URL and SUPABASE_ANON_KEY to the .env.
pnpm startVerify that no errors are logged and that the data is being written to the supabase db.
Database changes
None
Deployment instructions
N/A
Discord username
4n7m4n