feat(appwrite): add appwrite storage driver support#724
Open
Myshkouski wants to merge 3 commits intounjs:mainfrom
Open
feat(appwrite): add appwrite storage driver support#724Myshkouski wants to merge 3 commits intounjs:mainfrom
Myshkouski wants to merge 3 commits intounjs:mainfrom
Conversation
- Add new Appwrite storage driver implementation - Include configuration options and type definitions - Add environment variables for Appwrite integration - Include test suite for the new driver - Update package dependencies with node-appwrite and base-x
- Introduce dedicated utility module for Appwrite client management - Implement RequireAllOrNone type constraint for storage key configuration - Unify error handling patterns across all storage operations - Include driver context in API error reporting
Extract key encoding/decoding logic into reusable utilities and simplify test configuration by centralizing project options
Myshkouski
changed the title
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #724 +/- ##
=======================================
Coverage 91.77% 91.77%
=======================================
Files 5 5
Lines 401 401
Branches 126 127 +1
=======================================
Hits 368 368
Misses 33 33 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Resolves #722:
CI changes required:
VITE_APPWRITE_ENDPOINT,VITE_APPWRITE_PROJECT_ID,VITE_APPWRITE_BUCKET_IDandVITE_APPWRITE_API_KEYto Github environment variables for tests.