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

Add store abstraction #3

Open
Ozoniuss opened this issue Oct 14, 2023 · 0 comments
Open

Add store abstraction #3

Ozoniuss opened this issue Oct 14, 2023 · 0 comments

Comments

@Ozoniuss
Copy link
Owner

Ozoniuss commented Oct 14, 2023

Having the bussiness logic directly inside the handlers (including the database interaction) is not something I'm proud of. This is the most basic "transaction script" approach I started with because I wasn't really writing tests back then, and now testing is a pain in the ass. If I'm going to stick with the layered architecture I'd like to at least have an application layer on top of this.

Having the store abstraction would be reusable if I'm moving towards ports & adapters at some point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant