Merge pull request #142 from SumoLogic/ssain-user-resource-docs-changes

make is_active required and fix docs

Author: seansain

CHANGELOG.md

@@ -1,4 +1,8 @@
-## 2.6.4 (Unreleased)
+## 2.7.0 (Unreleased)
+
+BUG FIXES:
+
+* is_active is now required in sumologic_user, doc fixes
 
 ## 2.6.3 (January 15, 2021)
 

sumologic/resource_sumologic_user.go

@@ -50,7 +50,7 @@ func resourceSumologicUser() *schema.Resource {
 			},
 			"is_active": {
 				Type:     schema.TypeBool,
-				Optional: true,
+				Required: true,
 			},
 			"transfer_to": {
 				Type:     schema.TypeString,

website/docs/r/user.html.markdown

@@ -23,17 +23,17 @@ resource "sumologic_user" "example_user1" {
   first_name   = "Jon"
   last_name    = "Doe"
   email        = "[email protected]"
-  active       = false
-  role_ids     = ["${sumologic_role.example_role.id}"]
+  is_active    = true
+  role_ids     = [sumologic_role.example_role.id]
   transfer_to  = ""
 }
 
 resource "sumologic_user" "example_user2" {
   first_name   = "Jane"
   last_name    = "Smith"
   email        = "[email protected]"
-  role_ids     = ["${sumologic_role.example_role.id}"]
-  transfer_to  = "${sumologic_user.example_user1.id}"
+  role_ids     = [sumologic_role.example_role.id]
+  transfer_to  = sumologic_user.example_user1.id
 }
 ```
 
@@ -44,7 +44,7 @@ The following arguments are supported:
 - `first_name` - (Required) First name of the user.
 - `last_name` - (Required) Last name of the user.
 - `email` - (Required) Last name of the user.
-- `active` - (Optional) This has the value true if the user is active and false if they have been deactivated..
+- `is_active` - (Required) This has the value true if the user is active and false if they have been deactivated..
 - `role_ids` - (Required) List of roleIds associated with the user.
 - `transfer_to` - (Required) UserId of user to transfer this user's content to on deletion, can be empty. Must be applied prior to deletion to take effect.