-
Notifications
You must be signed in to change notification settings - Fork 11
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
feat(storage): add redis storage #63
Conversation
server/CompositeStorage/lib/index.ts
Outdated
Maybe | ||
} from '@dynamico/driver'; | ||
|
||
export class CompositeStorage implements Storage { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's a composition storage
export class CompositeStorage implements Storage { | |
export class CompositionStorage implements Storage { |
server/CompositeStorage/lib/index.ts
Outdated
ComponentTree, | ||
ComponentGetter, | ||
Maybe | ||
} from '@dynamico/driver'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we getting these types out of the driver and not the common types?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it's kind of weird... Not sure what's the correct structure here is but I wanted some package to export the types and not publish a @types/dynamico
package. I'll play with it some more to find the best setup...
saveComponent(component: Component, files: File[]): Promise<void>; | ||
} | ||
|
||
export type Storage = IndexStorage & ComponentsStorage; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beautiful
e196187
to
61a5229
Compare
c866ba6
to
e21d4f3
Compare
* re-organize types * add composite storage * tests for redis storage and composition storage
e21d4f3
to
6a949f6
Compare
Solves redis storage and a way to composite storages #48 and Common types should be exported from a shared package #17