@postmodern this kind of thing drives me crazy. I don't like the include Dep thing either.

I like where they went with Repositories but just don't like the integration of it.

2
Share
Share on Mastodon
Share on Twitter
Share on Facebook
Share on Linkedin
Ryan Bigg

@nateberkopec @postmodern Reckon that should just infer the table name from the class name? What about a syntax like MyApp::DB::Relation["posts"]?

0
21d
postmodern

@nateberkopec same. Also don't like `include Deps['posts.create_post']` and `create_post.call(...)`. We're already in the context of Post-related things. Also I think validations should be separate or somehow coupled with the create action; after all there could be multiple ways something gets created.

0
21d
Replies