Skip to content

Add custom_attributes field to SSO Profile#434

Merged
mattgd merged 3 commits intomainfrom
feature/dsync-2978-add-sso-profile-custom_attributes-to-python-sdk
Apr 25, 2025
Merged

Add custom_attributes field to SSO Profile#434
mattgd merged 3 commits intomainfrom
feature/dsync-2978-add-sso-profile-custom_attributes-to-python-sdk

Conversation

@mattgd
Copy link
Copy Markdown
Contributor

@mattgd mattgd commented Apr 17, 2025

Description

Add custom_attributes field to SSO Profile.

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.

@mattgd mattgd self-assigned this Apr 17, 2025
@linear
Copy link
Copy Markdown

linear bot commented Apr 17, 2025

first_name=None,
last_name=None,
profile=None,
role=None,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This was incorrect in fixture, just fixing.

role: Optional[ProfileRole] = None
groups: Optional[Sequence[str]] = None
raw_attributes: Mapping[str, Any]
custom_attributes: Optional[Mapping[str, Any]] = None
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'd like to add a generic type for custom_attributes that developers can override, but ran into some mypy limitations with default values for generic types. Keeping this as-is for now.

@mattgd mattgd marked this pull request as ready for review April 23, 2025 19:45
@mattgd mattgd requested a review from a team as a code owner April 23, 2025 19:45
@mattgd mattgd requested a review from faroceann April 23, 2025 19:45
@mattgd mattgd merged commit aa98f87 into main Apr 25, 2025
6 checks passed
@mattgd mattgd deleted the feature/dsync-2978-add-sso-profile-custom_attributes-to-python-sdk branch April 25, 2025 11:40
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