Skip to content

Move Organization operations to the top-level - #68

Merged
maxdeviant merged 1 commit into
masterfrom
move-organization-operations-to-top-level
May 12, 2021
Merged

maxdeviant merged 1 commit into
masterfrom
move-organization-operations-to-top-level

Conversation

@maxdeviant

Copy link
Copy Markdown
Contributor

This PR moves the Organization operations to their own top-level class in the SDK.

This is a breaking change that will be part of v1.0.0.

Resolves SDK-170.

@maxdeviant maxdeviant self-assigned this May 12, 2021
@maxdeviant
maxdeviant requested a review from a team as a code owner May 12, 2021 02:43
@linear

linear Bot commented May 12, 2021

Copy link
Copy Markdown

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #68 (0e64cbc) into master (f0629ff) will increase coverage by 0.22%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   92.22%   92.44%   +0.22%     
==========================================
  Files          16       17       +1     
  Lines         373      384      +11     
==========================================
+ Hits          344      355      +11     
  Misses         29       29              
Impacted Files Coverage Δ
workos/organizations.py 100.00% <100.00%> (ø)
workos/portal.py 100.00% <100.00%> (ø)
workos/utils/validation.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0629ff...0e64cbc. Read the comment docs.

@maxdeviant
maxdeviant merged commit e767631 into master May 12, 2021
@maxdeviant
maxdeviant deleted the move-organization-operations-to-top-level branch May 12, 2021 02:53
@maxdeviant maxdeviant mentioned this pull request May 12, 2021
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.

3 participants