search

firstdraft / appdev_support

MIT License
0 stars 0 forks source link

Feature requests #3

Open jelaniwoods opened 1 year ago

jelaniwoods commented 1 year ago

From the backlog of issues over the years that seem like patches that belong in this gem:

jelaniwoods commented 1 year ago

Maybe also better errors whitelisting

bpurinton commented 1 year ago

Adding from closed linear ticket:

Change printing of ActiveRecord_Relation class It’s so cumbersome to talk about e.g. Movie::ActiveRecord_Relation . I wonder if we can switch to saying/printing Relation::Movie, even though it’s not technically accurate. Then we can say “A Relation containing Movies” instead of “An ActiveRecord Relation containing Movies” Create a pryrc override for this? (e.g., https://github.com/firstdraft/appdev_support/pull/2)

bpurinton commented 2 weeks ago

Maybe we should override to_s for an AR instance the way we did for AR::Relation; e.g. instead of #<User 0x129048102589>, have something like #<User 0x10298319842 (one row from the users table)>

Better:

#<An instance of the User model representing row 42>

Since students often get stuck as soon as they see #<User 0x129...

Now that I'm looking at it, we already did some work along these lines. It's just not appearing, perhaps something to do with ActionView https://github.com/firstdraft/appdev_support?tab=readme-ov-file#usage