Remove deprecated code for major version (#1224)

Author: mattgd

src/audit-logs/audit-logs.spec.ts

@@ -225,7 +225,6 @@ describe('AuditLogs', () => {
 
         const options: AuditLogExportOptions = {
           actions: ['foo', 'bar'],
-          actors: ['Jon', 'Smith'],
           actorNames: ['Jon', 'Smith'],
           actorIds: ['user_foo', 'user_bar'],
           organizationId: 'org_123',

src/audit-logs/interfaces/audit-log-export-options.interface.ts

@@ -1,9 +1,5 @@
 export interface AuditLogExportOptions {
   actions?: string[];
-  /**
-   * @deprecated Please use `actorNames` instead.
-   */
-  actors?: string[];
   actorNames?: string[];
   actorIds?: string[];
   organizationId: string;
@@ -14,7 +10,6 @@ export interface AuditLogExportOptions {
 
 export interface SerializedAuditLogExportOptions {
   actions?: string[];
-  actors?: string[];
   actor_names?: string[];
   actor_ids?: string[];
   organization_id: string;

src/audit-logs/serializers/audit-log-export-options.serializer.ts

@@ -7,7 +7,6 @@ export const serializeAuditLogExportOptions = (
   options: AuditLogExportOptions,
 ): SerializedAuditLogExportOptions => ({
   actions: options.actions,
-  actors: options.actors,
   actor_names: options.actorNames,
   actor_ids: options.actorIds,
   organization_id: options.organizationId,

src/common/interfaces/event.interface.ts

@@ -403,22 +403,6 @@ export interface UserDeletedEventResponse extends EventResponseBase {
   data: UserResponse;
 }
 
-/**
- * @deprecated Use OrganizationMembershipCreated instead. Will be removed in a future major version.
- */
-export interface OrganizationMembershipAdded extends EventBase {
-  event: 'organization_membership.added';
-  data: OrganizationMembership;
-}
-
-/**
- * @deprecated Use OrganizationMembershipCreatedResponse instead. Will be removed in a future major version.
- */
-export interface OrganizationMembershipAddedResponse extends EventResponseBase {
-  event: 'organization_membership.added';
-  data: OrganizationMembershipResponse;
-}
-
 export interface OrganizationMembershipCreated extends EventBase {
   event: 'organization_membership.created';
   data: OrganizationMembership;
@@ -452,23 +436,6 @@ export interface OrganizationMembershipUpdatedResponse
   data: OrganizationMembershipResponse;
 }
 
-/**
- * @deprecated Use OrganizationMembershipDeleted instead. Will be removed in a future major version.
- */
-export interface OrganizationMembershipRemoved extends EventBase {
-  event: 'organization_membership.removed';
-  data: OrganizationMembership;
-}
-
-/**
- * @deprecated Use OrganizationMembershipDeletedResponse instead. Will be removed in a future major version.
- */
-export interface OrganizationMembershipRemovedResponse
-  extends EventResponseBase {
-  event: 'organization_membership.removed';
-  data: OrganizationMembershipResponse;
-}
-
 export interface OrganizationCreatedEvent extends EventBase {
   event: 'organization.created';
   data: Organization;
@@ -604,11 +571,9 @@ export type Event =
   | UserCreatedEvent
   | UserUpdatedEvent
   | UserDeletedEvent
-  | OrganizationMembershipAdded
   | OrganizationMembershipCreated
   | OrganizationMembershipDeleted
   | OrganizationMembershipUpdated
-  | OrganizationMembershipRemoved
   | RoleCreatedEvent
   | RoleDeletedEvent
   | RoleUpdatedEvent
@@ -653,11 +618,9 @@ export type EventResponse =
   | UserCreatedEventResponse
   | UserUpdatedEventResponse
   | UserDeletedEventResponse
-  | OrganizationMembershipAddedResponse
   | OrganizationMembershipCreatedResponse
   | OrganizationMembershipDeletedResponse
   | OrganizationMembershipUpdatedResponse
-  | OrganizationMembershipRemovedResponse
   | RoleCreatedEventResponse
   | RoleDeletedEventResponse
   | RoleUpdatedEventResponse

src/common/serializers/event.serializer.ts

@@ -141,11 +141,9 @@ export const deserializeEvent = (event: EventResponse): Event => {
         event: event.event,
         data: deserializeUser(event.data),
       };
-    case 'organization_membership.added':
     case 'organization_membership.created':
     case 'organization_membership.deleted':
     case 'organization_membership.updated':
-    case 'organization_membership.removed':
       return {
         ...eventBase,
         event: event.event,

src/events/events.spec.ts

@@ -25,7 +25,6 @@ describe('Event', () => {
       organizationId: 'org_1234',
       name: 'Connection',
       type: ConnectionType.OktaSAML,
-      connectionType: ConnectionType.OktaSAML,
       state: 'active',
       domains: [],
       createdAt: '2020-05-06 04:21:48.649164',

src/mfa/interfaces/index.ts

@@ -6,4 +6,3 @@ export * from './sms.interface';
 export * from './totp.interface';
 export * from './verify-challenge-options';
 export * from './verify-challenge-response';
-export * from './verify-factor-options';

src/mfa/interfaces/verify-factor-options.ts

@@ -1,17 +1,16 @@
 import { WorkOS } from '../workos';
 import {
-  ChallengeFactorOptions,
   Challenge,
+  ChallengeFactorOptions,
+  ChallengeResponse,
   EnrollFactorOptions,
   Factor,
+  FactorResponse,
+  FactorWithSecrets,
+  FactorWithSecretsResponse,
   VerifyChallengeOptions,
-  VerifyFactorOptions,
   VerifyResponse,
-  FactorResponse,
-  ChallengeResponse,
   VerifyResponseResponse,
-  FactorWithSecretsResponse,
-  FactorWithSecrets,
 } from './interfaces';
 import {
   deserializeChallenge,
@@ -73,13 +72,6 @@ export class Mfa {
     return deserializeChallenge(data);
   }
 
-  /**
-   * @deprecated Please use `verifyChallenge` instead.
-   */
-  async verifyFactor(options: VerifyFactorOptions): Promise<VerifyResponse> {
-    return this.verifyChallenge(options);
-  }
-
   async verifyChallenge(
     options: VerifyChallengeOptions,
   ): Promise<VerifyResponse> {

src/mfa/mfa.ts

@@ -6,31 +6,13 @@ export interface CreateOrganizationOptions {
   domainData?: DomainData[];
   externalId?: string | null;
   metadata?: Record<string, string>;
-
-  /**
-   * @deprecated If you need to allow sign-ins from any email domain, contact [email protected].
-   */
-  allowProfilesOutsideOrganization?: boolean;
-  /**
-   * @deprecated Use `domain_data` instead.
-   */
-  domains?: string[];
 }
 
 export interface SerializedCreateOrganizationOptions {
   name: string;
   domain_data?: DomainData[];
   external_id?: string | null;
   metadata?: Record<string, string>;
-
-  /**
-   * @deprecated If you need to allow sign-ins from any email domain, contact [email protected].
-   */
-  allow_profiles_outside_organization?: boolean;
-  /**
-   * @deprecated Use `domain_data` instead.
-   */
-  domains?: string[];
 }
 
 export type CreateOrganizationRequestOptions = Pick<