Skip to content
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

story(gRPC): event store constructor #59

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

erictg
Copy link
Member

@erictg erictg commented Nov 14, 2022

Closes #54

Do pr #52 first

@erictg erictg requested a review from Zaba505 November 14, 2022 14:31
@erictg erictg added grpc gRPC API related eventstore Event Store related enhancement New feature or request labels Nov 14, 2022
grpc/service.go Outdated Show resolved Hide resolved
grpc/service.go Outdated Show resolved Hide resolved
import "github.com/z5labs/evrys/eventstore"

// EventStore combines single method interfaces from eventstore into one interface
type EventStore interface {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think making a new file just for this is overkill

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file can be used for other interfaces. I like organizing interfaces in one place

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request eventstore Event Store related grpc gRPC API related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

story(grpc): support constructor config for event store implementation
2 participants