fix: remote_hook_id should use value from state (#71)

* fix: remote_hook_id should use value from state

Because remote_hook_id is a computed value, it should use the value from
state in update operation.

* remove commented code

Author: stanleyz

internal/provider/workspace_webhook_resource.go

@@ -150,12 +150,11 @@ func (r *WorkspaceWebhookResource) Create(ctx context.Context, req resource.Crea
 	plan.Branch.ElementsAs(ctx, &branchList, true)
 	plan.Path.ElementsAs(ctx, &pathList, true)
 	bodyRequest := &client.WorkspaceWebhookEntity{
-		ID:           uuid.New().String(),
-		Path:         strings.Join(pathList, ","),
-		Branch:       strings.Join(branchList, ","),
-		TemplateId:   plan.TemplateId.ValueString(),
-		RemoteHookId: plan.RemoteHookId.ValueString(),
-		Event:        plan.Event.ValueString(),
+		ID:         uuid.New().String(),
+		Path:       strings.Join(pathList, ","),
+		Branch:     strings.Join(branchList, ","),
+		TemplateId: plan.TemplateId.ValueString(),
+		Event:      plan.Event.ValueString(),
 	}
 
 	var out = new(bytes.Buffer)
@@ -279,7 +278,7 @@ func (r *WorkspaceWebhookResource) Update(ctx context.Context, req resource.Upda
 		Path:         strings.Join(pathList, ","),
 		Branch:       strings.Join(branchList, ","),
 		TemplateId:   plan.TemplateId.ValueString(),
-		RemoteHookId: plan.RemoteHookId.ValueString(),
+		RemoteHookId: state.RemoteHookId.ValueString(),
 		Event:        plan.Event.ValueString(),
 		ID:           state.ID.ValueString(),
 	}