-
Notifications
You must be signed in to change notification settings - Fork 44
Update Node SDK to ensure that all audit-log api's are in it #1457
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
base: main
Are you sure you want to change the base?
Conversation
|
@greptile review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
16 files reviewed, 1 comment
|
@greptile re-review plz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5 files reviewed, no comments
|
@greptile re-review plz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
6 files reviewed, no comments
|
@greptile re-review plz |
Greptile OverviewGreptile SummaryThis PR adds the List Schemas endpoint to the Audit Logs API, allowing users to retrieve all schemas for a specific audit log action with pagination support. Key changes:
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant Client
participant AuditLogs
participant WorkOS
participant API as WorkOS API
Client->>AuditLogs: listSchemas({action, ...paginationOptions})
AuditLogs->>AuditLogs: Extract action from options
AuditLogs->>AuditLogs: Build endpoint: /audit_logs/actions/{action}/schemas
AuditLogs->>WorkOS: fetchAndDeserialize(endpoint, deserializeListedAuditLogSchema, paginationOptions)
WorkOS->>API: GET /audit_logs/actions/{action}/schemas
API-->>WorkOS: ListResponse<ListAuditLogSchemaItemResponse>
WorkOS->>WorkOS: deserializeList(data, deserializeListedAuditLogSchema)
loop For each schema in response
WorkOS->>WorkOS: deserializeListedAuditLogSchema()
WorkOS->>WorkOS: Convert created_at to createdAt
WorkOS->>WorkOS: deserializeMetadata for targets, actor, metadata
end
WorkOS-->>AuditLogs: List<ListedAuditLogSchema>
AuditLogs->>AuditLogs: Create AutoPaginatable wrapper
AuditLogs-->>Client: AutoPaginatable<ListedAuditLogSchema>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
4 files reviewed, no comments
Description
Adding in the following Audit Log endpoints: