Usage of citext
instead of text
on all fields, and applying trim()
to those.
#1580
pranav-super
started this conversation in
Ideas
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
A minor issue that was encountered while debugging the
derivation_group
table led us to the realization that it might be worthwhile to enforce case-insensitivity and trimming on text fields.We had generated two derivation groups with similar names (the difference, presumably, being whitespace), which was completely undiscoverable on the frontend.
Furthermore, it is currently possible to allow the same name twice as long as the case varies, which @Mythicaeda suggested might be unideal. However, such a change would be sweeping across a lot of the database. Per @mattdailis and @Mythicaeda, we decided to open up a discussion on this!
(NOTE: That this issue was uncovered using derivation groups is unimportant, this issue of part of or the entirety of a primary key being text and possible overlap owing to whitespace is pretty generalizable across the AERIE backend...)
Beta Was this translation helpful? Give feedback.
All reactions