Skip to content

refactor: remove unused UpstashRedis default dbName#213

Open
droguljic wants to merge 2 commits into
docs/v2-updatefrom
refactor/remove-upstash-redis-defult-db-name
Open

refactor: remove unused UpstashRedis default dbName#213
droguljic wants to merge 2 commits into
docs/v2-updatefrom
refactor/remove-upstash-redis-defult-db-name

Conversation

@droguljic
Copy link
Copy Markdown
Contributor

@droguljic droguljic commented May 6, 2026

The component internally used the default value for dbName, but since the dbName property is required, the default is never used.

Depends on: #212

Documentation for v2, generated with the help of AI.
Component specific documentation is moved into their respective folders,
main `README.md` only references it.
The `examples` folder is for now removed and replaced partially by the in
documentation examples.
@droguljic droguljic added this to the v2 milestone May 6, 2026
@droguljic droguljic requested review from bornast and mandryllo May 6, 2026 12:25
@droguljic droguljic added the Don't merge Do not merge this PR label May 6, 2026
The component internally used the default value for `dbName`, but
since the `dbName` property is required, the default is never used.
@droguljic droguljic force-pushed the refactor/remove-upstash-redis-defult-db-name branch from 39b05f6 to 2c5ea62 Compare May 6, 2026 14:00
@droguljic droguljic changed the title refactor: remove UpstashRedis default dbName refactor: remove unused UpstashRedis default dbName May 6, 2026
@droguljic droguljic force-pushed the docs/v2-update branch 2 times, most recently from f5cf036 to e033356 Compare May 15, 2026 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Don't merge Do not merge this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants