Merge pull request #79 from DopplerHQ/rgharris/proj-memb-env-set

Ignore order of project_member environments
DopplerHQ · Mar 1, 2024 · 4aaa818 · 4aaa818
2 parents c101141 + 83a3fa2
commit 4aaa818
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/docs/resources/project_member_group.md b/docs/resources/project_member_group.md
@@ -32,7 +32,7 @@ resource "doppler_project_member_group" "backend_engineering" {
 
 ### Optional
 
-- `environments` (List of String) The environments in the project where this access will apply (null or omitted for roles with access to all environments)
+- `environments` (Set of String) The environments in the project where this access will apply (null or omitted for roles with access to all environments)
 
 ### Read-Only
 

diff --git a/docs/resources/project_member_service_account.md b/docs/resources/project_member_service_account.md
@@ -32,7 +32,7 @@ resource "doppler_project_member_service_account" "backend_ci" {
 
 ### Optional
 
-- `environments` (List of String) The environments in the project where this access will apply (null or omitted for roles with access to all environments)
+- `environments` (Set of String) The environments in the project where this access will apply (null or omitted for roles with access to all environments)
 
 ### Read-Only
 

diff --git a/doppler/resource_project_member.go b/doppler/resource_project_member.go
@@ -37,7 +37,7 @@ func (builder ResourceProjectMemberBuilder) Build() *schema.Resource {
 		},
 		"environments": {
 			Description: "The environments in the project where this access will apply (null or omitted for roles with access to all environments)",
-			Type:        schema.TypeList,
+			Type:        schema.TypeSet,
 			Optional:    true,
 			Elem: &schema.Schema{
 				Type: schema.TypeString,
@@ -67,7 +67,7 @@ func (builder ResourceProjectMemberBuilder) CreateContextFunc() schema.CreateCon
 		project := d.Get("project").(string)
 		role := d.Get("role").(string)
 
-		rawEnvironments := d.Get("environments").([]interface{})
+		rawEnvironments := d.Get("environments").(*schema.Set).List()
 		environments := make([]string, len(rawEnvironments))
 		for i, v := range rawEnvironments {
 			environments[i] = v.(string)
@@ -113,7 +113,7 @@ func (builder ResourceProjectMemberBuilder) UpdateContextFunc() schema.UpdateCon
 
 		var environments []string
 		if d.HasChange("environments") {
-			rawEnvironments := d.Get("environments").([]interface{})
+			rawEnvironments := d.Get("environments").(*schema.Set).List()
 			environments = make([]string, len(rawEnvironments))
 			for i, v := range rawEnvironments {
 				environments[i] = v.(string)