Would appreciate someone looking over these to make sure they're correct.

There's one todo that I couldn't quite figure out, which is the tw_columns.secure column. It might make more sense after I've read the source code.

CommentFileSizeAuthor
#1 tw-schema-documentation.patch7.16 KBwebchick
tw-schema-documentation.patch6.7 KBwebchick

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
StatusFileSize
new tw-schema-documentation.patch7.16 KB

Oops. Forgot a couple of straggling t()s. We made this change in 6.something-recent because realistically translators are never going to touch this text and it just horrifically skews the completion percentage of translations.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Needs review » Fixed

Committed with a few tweaks.

The secure column is a holdover (originally had UI attached to it like ignorecol) to indicate a column containing things like raw passwords or credit card numbers that should be treated with care. It remains unimplemented, but the idea is that it should be available at the data level in views (so the migrate module can access it) but a handler would be implemented to obfuscate it on display of views (e.g, xxxxxxxxxxxx79).

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.