search

oasis4hedev / salsa

Web app for creating a syllabus in an accessible HTML document, publishing to the Canvas LMS, and more.
https://syllabustool.com
Other
6 stars 5 forks source link

Use nil for empty strings concern. #518

Open opsaaaaa opened 5 years ago

opsaaaaa commented 5 years ago

Multiple tables and models allow for and contain columns with empty strings. anytime a model receives an empty string it should probably save it as nil.

This should be built as a concern and then included in each model that needs it. And maybe only for the columns that need it?