@erniemiller @tenderlove Definitely should call it out in the 4.2 docs, though. Wanna open a PR?
-
-
Replying to @sgrif
@sgrif@tenderlove Yeah, plan to — gimme a few — catching up on some stuff over here.2 replies 0 retweets 0 likes -
Replying to @erniemiller
@sgrif One problem: can't find any solid docs yet on minimal API for a Type object. Not keen on recommending users subclass Type::Value1 reply 0 retweets 0 likes -
Replying to @erniemiller
@sgrif …if only because it has precision/scale/limit and possibly other stuff that's not relevant for a domain object attribute1 reply 0 retweets 0 likes -
Replying to @erniemiller
@sgrif It looks like it might just be cast/serialize/deserialize?2 replies 0 retweets 0 likes -
Replying to @erniemiller
@sgrif Meh, scratch that. More involved than that. Suggestion: configuration of precision/scale/limit should be in a Numeric Value subclass3 replies 0 retweets 0 likes -
Replying to @erniemiller
@erniemiller Strings, dates, and more also use those. The issue is avoiding `respond_to?` if it's used outside of the class.1 reply 0 retweets 0 likes -
Replying to @erniemiller
@sgrif Sometimes it's better to even duplicate the code (remember it's not about duplicated code, but duplicated domain logic)2 replies 0 retweets 0 likes -
Replying to @erniemiller
@sgrif Anyway, I digress. I am not gonna make recs about the attribute api in the PR, since it is in flux atm. Just gonna mention the caveat1 reply 0 retweets 0 likes
@erniemiller Feedback is always appreciated.
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.