Skip to content

Commit

Permalink
fix: Add omitempty tag on 'domains_default_role_id' property for upda…
Browse files Browse the repository at this point in the history
…ting the instance organization settings

As the 'domains_default_role_id' is an optional property, we should use the
omitempty tag in order to end up with an empty string as the request body
  • Loading branch information
chanioxaris committed Dec 1, 2023
1 parent e733a9d commit 8fd0bbf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion clerk/instances.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ type UpdateOrganizationSettingsParams struct {
AdminDeleteEnabled *bool `json:"admin_delete_enabled,omitempty"`
DomainsEnabled *bool `json:"domains_enabled,omitempty"`
DomainsEnrollmentModes []string `json:"domains_enrollment_modes,omitempty"`
DomainsDefaultRoleID *string `json:"domains_default_role_id"`
DomainsDefaultRoleID *string `json:"domains_default_role_id,omitempty"`
}

func (s *InstanceService) UpdateOrganizationSettings(params UpdateOrganizationSettingsParams) (*OrganizationSettingsResponse, error) {
Expand Down

0 comments on commit 8fd0bbf

Please sign in to comment.