Merge pull request #6 from Canva/tajp-OrganizationIntegration-FixScop…

…esTypeCanva2 fix: use string array for scopes type
Canva · Sep 26, 2022 · 34975c1 · 34975c1
2 parents 6ae6811 + fbe3df1
commit 34975c1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/sentry/organization_integrations.go b/sentry/organization_integrations.go
@@ -25,7 +25,7 @@ type OrganizationIntegration struct {
 	Icon        string                          `json:"icon"`
 	DomainName  string                          `json:"domainName"`
 	AccountType string                          `json:"accountType"`
-	Scopes      *string                         `json:"scopes"`
+	Scopes      []string                       `json:"scopes"`
 	Status      string                          `json:"status"`
 	Provider    OrganizationIntegrationProvider `json:"provider"`
 

diff --git a/sentry/organization_integrations_test.go b/sentry/organization_integrations_test.go
@@ -24,7 +24,7 @@ func TestOrganizationIntegrationsService_List(t *testing.T) {
 				"icon": "https://avatars.githubusercontent.com/u/583231?v=4",
 				"domainName": "github.com/octocat",
 				"accountType": "Organization",
-				"scopes": null,
+				"scopes": ["read", "write"],
 				"status": "active",
 				"provider": {
 					"key": "github",
@@ -65,7 +65,7 @@ func TestOrganizationIntegrationsService_List(t *testing.T) {
 			Icon:        "https://avatars.githubusercontent.com/u/583231?v=4",
 			DomainName:  "github.com/octocat",
 			AccountType: "Organization",
-			Scopes:      nil,
+			Scopes:      []string{"read", "write"},
 			Status:      "active",
 			Provider: OrganizationIntegrationProvider{
 				Key:        "github",