Closed raghubetina closed 4 years ago
What do you think of changing the copy in active_record_relation_patch.rb from:
active_record_relation_patch.rb
"Array containing #{row_count} #{model} #{"record".pluralize(row_count)}"
to
"#{self.class} (array with #{row_count} #{model} instances inside)"
This will be part of our larger project to refer to relations as relations rather than collections, etc, and encourage frequent use of .class .
.class
(I purposely removed the smart pluralization because I want them to see the dissonance of "1 records inside".)
FIxed in #95
jelaniwoods
commented
4 years ago jelaniwoods
commented
4 years ago
What do you think of changing the copy in
active_record_relation_patch.rbfrom:to
This will be part of our larger project to refer to relations as relations rather than collections, etc, and encourage frequent use of
.class.(I purposely removed the smart pluralization because I want them to see the dissonance of "1 records inside".)