faustinoaq changed the topic of #amber to: Welcome to Amber Framework community! | https://amberframework.org | Developer happiness, productivity and bare metal performance | GH: https://github.com/amberframework | Docs: https://docs.amberframework.org | Gitter: https://gitter.im/amberframework/amber | IRC Logger: https://irclog.whitequark.org/amber | Amber::Server.start
<FromGitter> <Blacksmoke16> its also an issue that needs to be resolved in Amber itself imo, amber and granite are too tightly coupled
<FromGitter> <Blacksmoke16> but thats prob another discussion
<FromGitter> <andrewc910> Yeah you've told me. I wish Jennifer would support annotations but it's not on the roadmap. What business logic of granite is rolled into Amber? It's pretty easy to switch the between orms so on the surface it seems like they're decoupled pretty well?
<FromGitter> <Blacksmoke16> main thing that comes to mind is like `User.new user_params.validate!`
<FromGitter> <andrewc910> Hmm why is that considered coupled together? You're instantiating an object in your own project? The validate method is from the router. The only granite thing is the initialization logic? Or am I missing something?
<FromGitter> <Blacksmoke16> its a bit better now than it wass
<FromGitter> <Blacksmoke16> however granite used to handle converting string values to their proper types
<FromGitter> <Blacksmoke16> also not a big of just parsing values out of a `Hash(String, String)`
<FromGitter> <Blacksmoke16> i would be more in favor of having a specialized initializer that takes `HTTP::Params`
<FromGitter> <andrewc910> What about sanitizing params if you just passed in the whole params object?
<FromGitter> <Blacksmoke16> sanitize what?
_whitelogger has joined #amber
FromGitter has quit [Ping timeout: 245 seconds]
_whitelogger has joined #amber
_whitelogger has joined #amber