Skip to content

refactor: Add widgets:dsync:manage scope to WidgetScope type#1506

Merged
chaance merged 1 commit intomainfrom
chance/fix-widget-scopes-type
Feb 27, 2026
Merged

refactor: Add widgets:dsync:manage scope to WidgetScope type#1506
chaance merged 1 commit intomainfrom
chance/fix-widget-scopes-type

Conversation

@chaance
Copy link
Contributor

@chaance chaance commented Feb 27, 2026

Description

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.

@chaance chaance requested a review from a team as a code owner February 27, 2026 17:50
@chaance chaance requested a review from mthadley February 27, 2026 17:50
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 27, 2026

Greptile Summary

This PR adds a new scope widgets:dsync:manage to the WidgetScope union type. The change enables directory sync (dsync) widget functionality by allowing tokens to be scoped to dsync management operations. The implementation follows the existing pattern of other widget scopes and maintains consistency with the codebase's naming conventions.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change is minimal and only adds a single scope value to a TypeScript union type. No logic changes, no security concerns, and follows existing patterns perfectly.
  • No files require special attention

Important Files Changed

Filename Overview
src/widgets/interfaces/get-token.ts Added widgets:dsync:manage scope to WidgetScope union type

Last reviewed commit: 023a02e

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@chaance chaance changed the title Add widgets:dsync:manage scope to WidgetScope type refactor : Add widgets:dsync:manage scope to WidgetScope type Feb 27, 2026
@chaance chaance changed the title refactor : Add widgets:dsync:manage scope to WidgetScope type refactor: Add widgets:dsync:manage scope to WidgetScope type Feb 27, 2026
@chaance chaance merged commit 79f0d75 into main Feb 27, 2026
9 of 11 checks passed
@chaance chaance deleted the chance/fix-widget-scopes-type branch February 27, 2026 17:58
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.

2 participants