-
Notifications
You must be signed in to change notification settings - Fork 21
feat: adding redis APL #374
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
Merged
Merged
Changes from 5 commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
ed8a77d
feat: first commit with redis
JannikZed 874e1bb
feat: changeset added
JannikZed c6b584b
fix: redis version more broad and return package manager
JannikZed 52090dc
fix: using external redis client and adding some docs
JannikZed 02fb495
fix: testfile
JannikZed 45e409a
Merge branch 'main' of https://github.com/saleor/app-sdk into redis-apl
JannikZed f4f6167
fix: removing not needed mock client functions
JannikZed 3c27022
fix: set the redis state on mocked client connect and disconnect
JannikZed b7b6cd1
fix:: making the docs more clear. Connect not needed manually.
JannikZed a078efb
fix: no longer use the process.env for the cache key
JannikZed 2c24b25
fix: ading redis to dev dependencies
JannikZed b555ac1
feat: real client test
JannikZed 0c56265
fix: using only redis sub methods actually needed
JannikZed 2df0a40
fix: integrate redis in tsup build pipeline
JannikZed 4dd1568
feat: adding integration test for redis
JannikZed 55a4c56
docs: adding docs on integration test
JannikZed 3fa3866
fix: type checking is failing so we need to use the src
JannikZed File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@saleor/app-sdk": minor | ||
| --- | ||
|
|
||
| Adding REDIS as APL provider |
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.