feat: Fix patch to put to update dashboards in blueprints

Nishu Goel · Nishu Goel · commit ce002ae92284 · 2025-06-03T14:59:09.000+02:00
diff --git a/.speakeasy/gen.lock b/.speakeasy/gen.lock
@@ -1,12 +1,12 @@
 lockVersion: 2.0.0
 id: 5a857039-7f4b-42d5-86fd-449767242ed2
 management:
-  docChecksum: 5c72a9268be23452215b09edbe6f57e7
+  docChecksum: dd5eb608fd0e0ea556a472c6122d83a7
   docVersion: 0.0.1
   speakeasyVersion: 1.477.0
   generationVersion: 2.497.0
-  releaseVersion: 0.15.1
-  configChecksum: fd7797f410ce9414dce5af84c5518410
+  releaseVersion: 0.15.2
+  configChecksum: 6cf307147e8ddd1dcd9dba2d693aeace
   repoURL: https://github.com/epilot-dev/terraform-provider-epilot-product.git
   repoSubDirectory: .
   published: true
diff --git a/README.md b/README.md
@@ -20,7 +20,6 @@ Dashboard API: API to store the dashboard configuration for the epilot 360 dashb
 ## Table of Contents
 <!-- $toc-max-depth=2 -->
 * [epilot-dashboard](#epilot-dashboard)
-  * [🏗 **Welcome to your new Terraform Provider!** 🏗](#welcome-to-your-new-terraform-provider)
   * [Installation](#installation)
   * [Available Resources and Data Sources](#available-resources-and-data-sources)
   * [Testing the provider locally](#testing-the-provider-locally)
@@ -39,7 +38,7 @@ terraform {
   required_providers {
     epilot-dashboard = {
       source  = "epilot-dev/epilot-dashboard"
-      version = "0.15.1"
+      version = "0.15.2"
     }
   }
 }
diff --git a/docs/index.md b/docs/index.md
@@ -17,7 +17,7 @@ terraform {
   required_providers {
     epilot-dashboard = {
       source  = "epilot-dev/epilot-dashboard"
-      version = "0.15.1"
+      version = "0.15.2"
     }
   }
 }
diff --git a/docs/resources/dashboard.md b/docs/resources/dashboard.md
@@ -25,12 +25,12 @@ resource "epilot-dashboard_dashboard" "my_dashboard" {
 
 ### Required
 
-- `tiles` (String) Requires replacement if changed.; Parsed as JSON.
-- `title` (String) Requires replacement if changed.
+- `tiles` (String) Parsed as JSON.
+- `title` (String)
 
 ### Optional
 
-- `id` (String) Unique identifier for dashboard. Requires replacement if changed.
+- `id` (String) Unique identifier for dashboard
 
 ## Import
 
diff --git a/examples/README.md b/examples/README.md
@@ -0,0 +1 @@
+# TODO
diff --git a/examples/provider/provider.tf b/examples/provider/provider.tf
@@ -2,7 +2,7 @@ terraform {
   required_providers {
     epilot-dashboard = {
       source  = "epilot-dev/epilot-dashboard"
-      version = "0.15.1"
+      version = "0.15.2"
     }
   }
 }
diff --git a/examples/resources/epilot-dashboard_dashboard/resource.tf b/examples/resources/epilot-dashboard_dashboard/resource.tf
@@ -2,24 +2,4 @@ resource "epilot-dashboard_dashboard" "my_dashboard" {
   id    = "3fa85f64-5717-4562-b3fc-2c963f66afa6"
   tiles = "{ \"see\": \"documentation\" }"
   title = "Employee Dashboard"
-}
-
-terraform {
-  required_providers {
-    epilot-dashboard = {
-      source  = "epilot-dev/epilot-dashboard"
-      version = "0.15.1"
-    }
-  }
-}
-
-provider "epilot-dashboard" {
-  # Configuration options
-
-  epilot_auth="eyJraWQiOiJ2ZFR0MGQrK1RMc2FQZ2tsQ3AzMDVGbEMxc1lOUCtUOXpsaElzMkJ3WERrPSIsImFsZyI6IlJTMjU2In0.eyJzdWIiOiIxNzEyMTkwMy1kM2JlLTRhZTktODZiZS04YjhkZDRmYzY0ZTYiLCJlbWFpbF92ZXJpZmllZCI6dHJ1ZSwiaXNzIjoiaHR0cHM6XC9cL2NvZ25pdG8taWRwLmV1LWNlbnRyYWwtMS5hbWF6b25hd3MuY29tXC9ldS1jZW50cmFsLTFfaGh6MnVJQ2xIIiwicGhvbmVfbnVtYmVyX3ZlcmlmaWVkIjp0cnVlLCJjdXN0b206aXZ5X29yZ19pZCI6IjY2IiwiY29nbml0bzp1c2VybmFtZSI6Im4uZ29lbEBlcGlsb3QuY2xvdWQiLCJjdXN0b206aXZ5X3VzZXJfaWQiOiI4MjYwMiIsImF1ZCI6ImdqOXAwanJlaWh0cTAwY3JpNmEwZmUzMDYiLCJldmVudF9pZCI6IjIwNzM0YjM3LTU5ZjctNDIwYS05NGNhLTcyY2Y0MGVlMmRiNiIsInRva2VuX3VzZSI6ImlkIiwiYXV0aF90aW1lIjoxNzQ4ODc5MDk4LCJleHAiOjE3NDg5NDY2NTAsImlhdCI6MTc0ODk0MzA1MCwiZW1haWwiOiJuLmdvZWxAZXBpbG90LmNsb3VkIn0.lMMfJRH9T_7Lk5Gb1oVXgWlYCwIjZMxRyee3H7uSTBI_yxq7QDenPEmRRVyPdQIkNS15YepJ2-7r64Ny68Er0hYhbAuLFnd3KPUjlcUMwHnkgg-7Q0wcyWu1xwrLV7du0tPMZi_XgovJrXVY1PLVV_bY4q4AqZE6R2TUE4sPS9S6FmwSa6HfQbIUK55DrS3I5duAWIYAIdWEyNpekvXCC9FIT2iiBmUoEW4m7P-e_SzWpfbqlY5bpMVoeXsQs0E8x86sOxUxADJeR62_YVjgsSaVuwJgpdsrHoDFPQbK3h-hzALFz_MT97S_Dt2oh5v8DSI32L_NB-uid248cRCLOA"
-}
-
-# import {
-#   to = epilot-dashboard_dashboard.my_dashboard
-#   id = "5bc69ec6-b709-4b9c-865f-a3a07375005e"
-# }
+}
diff --git a/gen.yaml b/gen.yaml
@@ -28,7 +28,7 @@ go:
   outputModelSuffix: output
   packageName: openapi
 terraform:
-  version: 0.15.1
+  version: 0.15.2
   additionalDataSources: []
   additionalDependencies: {}
   additionalResources: []
diff --git a/internal/provider/dashboard_resource.go b/internal/provider/dashboard_resource.go
@@ -5,15 +5,12 @@ package provider
 import (
 	"context"
 	"fmt"
-	speakeasy_stringplanmodifier "github.com/epilot-dev/terraform-provider-epilot-dashboard/internal/planmodifiers/stringplanmodifier"
 	"github.com/epilot-dev/terraform-provider-epilot-dashboard/internal/sdk"
 	"github.com/epilot-dev/terraform-provider-epilot-dashboard/internal/sdk/models/operations"
 	"github.com/epilot-dev/terraform-provider-epilot-dashboard/internal/validators"
 	"github.com/hashicorp/terraform-plugin-framework/path"
 	"github.com/hashicorp/terraform-plugin-framework/resource"
 	"github.com/hashicorp/terraform-plugin-framework/resource/schema"
-	"github.com/hashicorp/terraform-plugin-framework/resource/schema/planmodifier"
-	"github.com/hashicorp/terraform-plugin-framework/resource/schema/stringplanmodifier"
 	"github.com/hashicorp/terraform-plugin-framework/schema/validator"
 	"github.com/hashicorp/terraform-plugin-framework/types"
 	"github.com/hashicorp/terraform-plugin-framework/types/basetypes"
@@ -48,32 +45,19 @@ func (r *DashboardResource) Schema(ctx context.Context, req resource.SchemaReque
 		MarkdownDescription: "Dashboard Resource",
 		Attributes: map[string]schema.Attribute{
 			"id": schema.StringAttribute{
-				Computed: true,
-				Optional: true,
-				PlanModifiers: []planmodifier.String{
-					stringplanmodifier.RequiresReplaceIfConfigured(),
-					speakeasy_stringplanmodifier.SuppressDiff(speakeasy_stringplanmodifier.ExplicitSuppress),
-				},
-				Description: `Unique identifier for dashboard. Requires replacement if changed.`,
+				Computed:    true,
+				Optional:    true,
+				Description: `Unique identifier for dashboard`,
 			},
 			"tiles": schema.StringAttribute{
-				Required: true,
-				PlanModifiers: []planmodifier.String{
-					stringplanmodifier.RequiresReplaceIfConfigured(),
-					speakeasy_stringplanmodifier.SuppressDiff(speakeasy_stringplanmodifier.ExplicitSuppress),
-				},
-				Description: `Requires replacement if changed.; Parsed as JSON.`,
+				Required:    true,
+				Description: `Parsed as JSON.`,
 				Validators: []validator.String{
 					validators.IsValidJSON(),
 				},
 			},
 			"title": schema.StringAttribute{
 				Required: true,
-				PlanModifiers: []planmodifier.String{
-					stringplanmodifier.RequiresReplaceIfConfigured(),
-					speakeasy_stringplanmodifier.SuppressDiff(speakeasy_stringplanmodifier.ExplicitSuppress),
-				},
-				Description: `Requires replacement if changed.`,
 			},
 		},
 	}
@@ -213,7 +197,36 @@ func (r *DashboardResource) Update(ctx context.Context, req resource.UpdateReque
 		return
 	}
 
-	// Not Implemented; all attributes marked as RequiresReplace
+	dashboard := data.ToSharedDashboard()
+	var id string
+	id = data.ID.ValueString()
+
+	request := operations.PutDashboardRequest{
+		Dashboard: dashboard,
+		ID:        id,
+	}
+	res, err := r.client.Dashboards.PutDashboard(ctx, request)
+	if err != nil {
+		resp.Diagnostics.AddError("failure to invoke API", err.Error())
+		if res != nil && res.RawResponse != nil {
+			resp.Diagnostics.AddError("unexpected http request/response", debugResponse(res.RawResponse))
+		}
+		return
+	}
+	if res == nil {
+		resp.Diagnostics.AddError("unexpected response from API", fmt.Sprintf("%v", res))
+		return
+	}
+	if res.StatusCode != 200 {
+		resp.Diagnostics.AddError(fmt.Sprintf("unexpected response from API. Got an unexpected response code %v", res.StatusCode), debugResponse(res.RawResponse))
+		return
+	}
+	if !(res.Dashboard != nil) {
+		resp.Diagnostics.AddError("unexpected response from API. Got an unexpected response body", debugResponse(res.RawResponse))
+		return
+	}
+	data.RefreshFromSharedDashboard(res.Dashboard)
+	refreshPlan(ctx, plan, &data, resp.Diagnostics)
 
 	// Save updated data into Terraform state
 	resp.Diagnostics.Append(resp.State.Set(ctx, &data)...)
diff --git a/internal/sdk/sdk.go b/internal/sdk/sdk.go
@@ -149,9 +149,9 @@ func New(opts ...SDKOption) *SDK {
 		sdkConfiguration: sdkConfiguration{
 			Language:          "go",
 			OpenAPIDocVersion: "0.0.1",
-			SDKVersion:        "0.15.1",
+			SDKVersion:        "0.15.2",
 			GenVersion:        "2.497.0",
-			UserAgent:         "speakeasy-sdk/terraform 0.15.1 2.497.0 0.0.1 github.com/epilot-dev/terraform-provider-epilot-dashboard/internal/sdk",
+			UserAgent:         "speakeasy-sdk/terraform 0.15.2 2.497.0 0.0.1 github.com/epilot-dev/terraform-provider-epilot-dashboard/internal/sdk",
 			Hooks:             hooks.New(),
 		},
 	}
diff --git a/overlay.yaml b/overlay.yaml
@@ -23,7 +23,7 @@ actions:
   - target: $["paths"]["/v1/dashboard/dashboards/{id}"]["get"]
     update:
       x-speakeasy-entity-operation: Dashboard#get
-  - target: $["paths"]["/v1/dashboard/dashboards/{id}"]["patch"]
+  - target: $["paths"]["/v1/dashboard/dashboards/{id}"]["put"]
     update:
       x-speakeasy-entity-operation: Dashboard#update
   - target: $["paths"]["/v1/dashboard/dashboards/{id}"]["delete"]

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,6 @@ Dashboard API: API to store the dashboard configuration for the epilot 360 dashb`
`20`	`20`	`## Table of Contents`
`21`	`21`	`<!-- $toc-max-depth=2 -->`
`22`	`22`	`* [epilot-dashboard](#epilot-dashboard)`
`23`		`- * [🏗 Welcome to your new Terraform Provider! 🏗](#welcome-to-your-new-terraform-provider)`
`24`	`23`	`* [Installation](#installation)`
`25`	`24`	`* [Available Resources and Data Sources](#available-resources-and-data-sources)`
`26`	`25`	`* [Testing the provider locally](#testing-the-provider-locally)`
`@@ -39,7 +38,7 @@ terraform {`
`39`	`38`	`required_providers {`
`40`	`39`	`epilot-dashboard = {`
`41`	`40`	`source = "epilot-dev/epilot-dashboard"`
`42`		`- version = "0.15.1"`
	`41`	`+ version = "0.15.2"`
`43`	`42`	`}`
`44`	`43`	`}`
`45`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ terraform {`
`17`	`17`	`required_providers {`
`18`	`18`	`epilot-dashboard = {`
`19`	`19`	`source = "epilot-dev/epilot-dashboard"`
`20`		`- version = "0.15.1"`
	`20`	`+ version = "0.15.2"`
`21`	`21`	`}`
`22`	`22`	`}`
`23`	`23`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@ terraform {`
`2`	`2`	`required_providers {`
`3`	`3`	`epilot-dashboard = {`
`4`	`4`	`source = "epilot-dev/epilot-dashboard"`
`5`		`- version = "0.15.1"`
	`5`	`+ version = "0.15.2"`
`6`	`6`	`}`
`7`	`7`	`}`
`8`	`8`	`}`