I defined it starting with -1 to keep 0 (no_restrictions
) as the default, and to preserve the logical order of restricting two factor authentication. @yorickpeterse if I change the enum to start at 0, do you think it would make more sense to use the following?
enum two_factor_auth_restriction: {
no_restrictions: 0,
disabled: 1,
required: 2
}
Brianna Wegryn (60d0dadf) at 06 Dec 21:42
Provide option to disable two-factor authentication
Brianna Wegryn (6a03fe7b) at 06 Dec 17:56
Provide option to disable two-factor authentication
@DouweM I believe I followed the steps (on that page) for doing the migration without requiring downtime.. Is there a specific line you are concerned about?
Brianna Wegryn (78d1031a) at 06 Dec 17:37
Provide option to disable two-factor authentication
... and 3587 more commits
@mydigitalself and @DouweM I am working on the next batch of requested changes now. Sorry for the delay!
Brianna Wegryn (f863808c) at 04 Dec 15:54
Merge branch 'gollum-initializer-warning' into 'master'
... and 58 more commits
Brianna Wegryn (fee2a543) at 30 Nov 18:59
Merge branch 'zj-blobs-bulk-gitaly' into 'master'
... and 3152 more commits