Skip to content

feat: add external_id support for organization create/update operations#90

Merged
gjtorikian merged 1 commit intoworkos:mainfrom
totalite:feat/external-id-create-update
Feb 11, 2026
Merged

feat: add external_id support for organization create/update operations#90
gjtorikian merged 1 commit intoworkos:mainfrom
totalite:feat/external-id-create-update

Conversation

@basbz
Copy link
Contributor

@basbz basbz commented Feb 11, 2026

Add external_id parameter support to create and update operations:

  • WorkOS.Organizations.create_organization/1,2 — accepts :external_id in opts
  • WorkOS.Organizations.update_organization/2,3 — accepts :external_id in opts

This is a follow-up to #83 which added external_id to the structs and added lookup-by-external-id endpoints. With this PR, the SDK fully supports the external_id field across all CRUD operations for Organizations.dd

…tion

Add `external_id` parameter support to create and update operations:
- `WorkOS.Organizations.create_organization/1,2` — accepts `:external_id` in opts
- `WorkOS.Organizations.update_organization/2,3` — accepts `:external_id` in opts

This is a follow-up to workos#83 which added `external_id` to the structs and
added lookup-by-external-id endpoints. With this PR, the SDK fully supports
the `external_id` field across all CRUD operations for Organizations.dd
@greptile-apps
Copy link

greptile-apps bot commented Feb 11, 2026

Greptile Overview

Greptile Summary

This PR completes the external_id support for Organizations by adding the parameter to create and update operations. The implementation follows the existing codebase patterns and complements PR #83 which added the struct field and lookup-by-external-id endpoints.

  • Added external_id parameter to create_organization/1,2 and update_organization/2,3
  • Updated documentation to describe the new parameter
  • Added comprehensive tests validating the new functionality
  • Updated mock client to include external_id in responses

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The changes are straightforward parameter additions that follow existing patterns in the codebase. The implementation is consistent with how other optional parameters are handled, includes proper documentation, and has comprehensive test coverage.
  • No files require special attention

Important Files Changed

Filename Overview
lib/workos/organizations.ex Added external_id parameter to create and update operations with proper documentation
test/support/organizations_client_mock.ex Updated mock responses to include external_id field for create and update operations
test/workos/organizations_test.exs Added comprehensive tests for external_id support in create and update operations

Copy link

@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.

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copy link
Contributor

@gjtorikian gjtorikian left a comment

Choose a reason for hiding this comment

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

thank you~

@gjtorikian gjtorikian merged commit d616563 into workos:main Feb 11, 2026
5 checks passed
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