-
Notifications
You must be signed in to change notification settings - Fork 18
IDE & Typehint improvements #264
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
Changes from all commits
8b9221f
8988129
6573b8e
92bc0c6
eb67567
0c5672a
ff55574
09c8949
829e418
f2d7f7b
d366c83
1ec12f4
7381751
19e68d1
52dc0ba
b25b57d
053ff55
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -108,11 +108,12 @@ public function challengeFactor( | |
|
|
||
|
|
||
| /** | ||
| * @deprecated 1.12.0 Use `verifyChallenge` instead. This method will be removed in a future major version. | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Incorrect ordering of @ attrs. |
||
| * Verifies the one time password provided by the end-user. | ||
| * | ||
| * @param string $authenticationChallengeId - The ID of the authentication challenge that provided the user the verification code. | ||
| * @param string $code - The verification code sent to and provided by the end user. | ||
| */ | ||
| */ | ||
|
|
||
| public function verifyFactor( | ||
| $authenticationChallengeId, | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -4,8 +4,16 @@ | |
|
|
||
| /** | ||
| * Class OrganizationMembership. | ||
| * | ||
| * @property 'organization_membership' $object | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ??? |
||
| * @property string $id | ||
| * @property string $userId | ||
| * @property string $organizationId | ||
| * @property RoleResponse $role | ||
| * @property 'active'|'inactive'|'pending' $status | ||
| * @property string $createdAt | ||
| * @property string $updatedAt | ||
| */ | ||
|
|
||
| class OrganizationMembership extends BaseWorkOSResource | ||
| { | ||
| public const RESOURCE_TYPE = "organization_membership"; | ||
|
|
@@ -15,6 +23,7 @@ class OrganizationMembership extends BaseWorkOSResource | |
| "id", | ||
| "userId", | ||
| "organizationId", | ||
| "role", | ||
| "status", | ||
| "createdAt", | ||
| "updatedAt" | ||
|
|
@@ -25,6 +34,7 @@ class OrganizationMembership extends BaseWorkOSResource | |
| "id" => "id", | ||
| "user_id" => "userId", | ||
| "organization_id" => "organizationId", | ||
| "role" => "role", | ||
| "status" => "status", | ||
| "created_at" => "createdAt", | ||
| "updated_at" => "updatedAt" | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,15 +6,74 @@ | |
| * Class Webhook. | ||
| * | ||
| * Representation of a webhook resulting from a client ConstructEvent function. | ||
| * | ||
| * @property-read 'user_registration_action_context'|'authentication_action_context' $object The type of webhook event | ||
| * | ||
| * User Registration Action Properties | ||
| * @property-read ?object{ | ||
| * object: 'user_data', | ||
| * email: string, | ||
| * first_name: string, | ||
| * last_name: string | ||
| * } $user_data User information for registration events | ||
|
Comment on lines
+13
to
+18
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes! This kind of array shape. |
||
| * @property-read ?object{ | ||
| * object: 'invitation', | ||
| * id: string, | ||
| * email: string, | ||
| * expires_at: string, | ||
| * created_at: string, | ||
| * updated_at: string, | ||
| * accepted_at: ?string, | ||
| * revoked_at: ?string, | ||
| * organization_id: string, | ||
| * inviter_user_id: string | ||
| * } $invitation Invitation details for registration events | ||
| * | ||
| * Authentication Action Properties | ||
| * @property-read ?object{ | ||
| * object: 'user', | ||
| * id: string, | ||
| * email: string, | ||
| * first_name: string, | ||
| * last_name: string, | ||
| * email_verified: bool, | ||
| * profile_picture_url: string, | ||
| * created_at: string, | ||
| * updated_at: string | ||
| * } $user User information for authentication events | ||
| * @property-read ?string $issuer The authentication issuer | ||
| * @property-read ?object{ | ||
| * object: 'organization', | ||
| * id: string, | ||
| * name: string, | ||
| * allow_profiles_outside_organization: bool, | ||
| * domains: array<string>, | ||
| * created_at: string, | ||
| * updated_at: string | ||
| * } $organization Organization details for authentication events | ||
| * @property-read ?object{ | ||
| * object: 'organization_membership', | ||
| * id: string, | ||
| * user_id: string, | ||
| * organization_id: string, | ||
| * role: array{slug: string}, | ||
| * status: string, | ||
| * created_at: string, | ||
| * updated_at: string | ||
| * } $organization_membership Organization membership details for authentication events | ||
| * | ||
| * Common Properties | ||
| * @property-read string $ip_address IP address of the event | ||
| * @property-read string $user_agent User agent string of the event | ||
| * @property-read string $device_fingerprint Device fingerprint of the event | ||
| */ | ||
| class Webhook | ||
| { | ||
| /** | ||
| * Creates a webhook object from a payload. | ||
| * | ||
| * @param $payload | ||
| * | ||
| * @return Webhook | ||
| * @param string $payload JSON string containing webhook data | ||
| * @return static | ||
| */ | ||
| public static function constructFromPayload($payload) | ||
| { | ||
|
|
||
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.
Using array shapes would improve this even more.