search

AllYourBot / hostedgpt

An open version of ChatGPT you can host anywhere or run locally.
MIT License
217 stars 85 forks source link

Delegate email to person #197

Closed olimart closed 2 months ago

olimart commented 2 months ago

Allow calling User.first.email

krschacht commented 2 months ago

Good call, @olimart could you add a single test to user_test.rb for this?

olimart commented 2 months ago

I could but have the impression than it will test the framework more than anything else. Let me know.

krschacht commented 2 months ago

Yea, it’s fine as it is. It’s a borderline one for me. I always add a simple test for associations and I find that really helpful when I’m refactoring a model — that way I can simply re-run the model’s test suite and I know if the basic assumptions are all still passing. But I don’t write tests for the basic attributes. Anyway, borderline. I’m merging it now.