Skip to content

Commit

Permalink
[AC-1260] Removed ssoRequired field from the sso details response (#5123
Browse files Browse the repository at this point in the history
)

* Removed ssoRequired field from the sso details response

* Fixed PR comment
  • Loading branch information
gbubemismith committed Apr 3, 2023
1 parent a68631c commit a78ed4c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
7 changes: 2 additions & 5 deletions apps/web/src/auth/sso.component.ts
Original file line number Diff line number Diff line change
Expand Up @@ -104,11 +104,8 @@ export class SsoComponent extends BaseSsoComponent {
const errorResponse: ErrorResponse = error as ErrorResponse;
switch (errorResponse.statusCode) {
case HttpStatusCode.NotFound:
if (errorResponse?.message?.includes("Claimed org domain not found")) {
// Do nothing. This is a valid case.
return;
}
break;
//this is a valid case for a domain not found
return;

default:
this.validationService.showError(errorResponse);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ export class OrganizationDomainSsoDetailsResponse extends BaseResponse {
organizationIdentifier: string;
ssoAvailable: boolean;
domainName: string;
ssoRequired: boolean;
verifiedDate?: Date;

constructor(response: any) {
Expand All @@ -14,7 +13,6 @@ export class OrganizationDomainSsoDetailsResponse extends BaseResponse {
this.organizationIdentifier = this.getResponseProperty("organizationIdentifier");
this.ssoAvailable = this.getResponseProperty("ssoAvailable");
this.domainName = this.getResponseProperty("domainName");
this.ssoRequired = this.getResponseProperty("ssoRequired");
this.verifiedDate = this.getResponseProperty("verifiedDate");
}
}

0 comments on commit a78ed4c

Please sign in to comment.