Skip to content

rename and deprecate the AuthorizationURLOptions interfaces #1282

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 27, 2025

Conversation

nicknisi
Copy link
Contributor

@nicknisi nicknisi commented May 21, 2025

Description

This pull request introduces changes to standardize and clarify the naming of authorization URL options interfaces across the SSO and User Management modules. It also includes minor import adjustments and code cleanup to align with the updated interface names.

Interface Renaming and Deprecation:

  • Renamed AuthorizationURLOptions to SSOAuthorizationURLOptions in the SSO module and marked the original interface as deprecated for backward compatibility. (src/sso/interfaces/authorization-url-options.interface.ts: [1] [2]
  • Renamed AuthorizationURLOptions to UserManagementAuthorizationURLOptions in the User Management module to better reflect its context. (src/user-management/interfaces/authorization-url-options.interface.ts: src/user-management/interfaces/authorization-url-options.interface.tsL1-R1)

Code Updates to Use New Interface Names:

  • Updated the SSO class to use SSOAuthorizationURLOptions instead of the deprecated AuthorizationURLOptions. (src/sso/sso.ts: src/sso/sso.tsL76-R76)
  • Updated the UserManagement class to use UserManagementAuthorizationURLOptions instead of the old AuthorizationURLOptions. (src/user-management/user-management.ts: src/user-management/user-management.tsL997-R997)

Import Adjustments and Cleanup:

  • Adjusted imports in src/sso/sso.ts and src/user-management/user-management.ts to reflect the renamed interfaces and remove unused imports. (src/sso/sso.ts: [1] [2]; src/user-management/user-management.ts: [3] [4]

These changes improve code clarity and maintainability by aligning interface names with their respective modules and ensuring backward compatibility where necessary.

image

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

fixes #1281

@nicknisi nicknisi requested a review from nholden May 27, 2025 16:36
@nicknisi nicknisi merged commit 423154e into main May 27, 2025
5 checks passed
@nicknisi nicknisi deleted the nicknisi/1281-identically-named-interfaces branch May 27, 2025 18:24
stacurry added a commit that referenced this pull request May 30, 2025
## Description
version bump that includes:
- #1282
- #1288

## Documentation

Does this require changes to the WorkOS Docs? E.g. the [API
Reference](https://workos.com/docs/reference) or code snippets need
updates.

```
[ ] Yes
```

If yes, link a related docs PR and add a docs maintainer as a reviewer.
Their approval is required.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Identically named interfaces with different shapes
3 participants