Skip to content

Commit 43d95a0

Browse files
chrisbobbegnprice
authored andcommitted
api [nfc]: Comment on GetServerSettingsResult.authenticationMethods
1 parent 31de114 commit 43d95a0

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

lib/api/route/realm.dart

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,15 @@ Future<GetServerSettingsResult> getServerSettings({
3030

3131
@JsonSerializable(fieldRename: FieldRename.snake)
3232
class GetServerSettingsResult {
33+
// This is marked as deprecated, but we still need it to answer some
34+
// questions:
35+
// - Do we offer dev login? (Yes if it has `dev: true`.)
36+
// - Do we offer password login? (Yes if it has `password: true` or `ldap: true`.)
37+
// - Do we offer a "Log in with SSO" button? (Yes if it has `remoteuser: true`.)
38+
//
39+
// Discussion: https://chat.zulip.org/#narrow/stream/378-api-design/topic/.60authentication_methods.60.20in.20server.20settings/near/1722986
3340
final Map<String, bool> authenticationMethods;
41+
3442
final List<ExternalAuthenticationMethod> externalAuthenticationMethods;
3543

3644
final int zulipFeatureLevel;

0 commit comments

Comments
 (0)