Merge pull request #16 from statuspal/fix/error-when-trying-to-change-description-from-something-to-null-or-empty-string

FIX: Error when trying to change an attribute from "something" to null or empty string.

Author: FabianDaniel00

CHANGELOG.md

@@ -5,6 +5,21 @@ All notable changes of the StatusPal Terraform provider will be documented in th
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.2.12] - 2024-10-17
+
+### Fixed
+
+- Error when trying to change an attribute from "something" to `null` or empty string.
+
+### Removed
+
+- `theme_configs` attribute from status page resource and data source, as it was deprecated.
+
+### Added
+
+- `zoom_notifications_enabled` and `allowed_email_domains` attributes to status page resource and data source.
+- `monitoring_options` attribute to service resource and data source.
+
 ## [0.2.11] - 2024-10-07
 
 ### Fixed

docs/data-sources/services.md

@@ -34,10 +34,6 @@ data "statuspal_services" "all" {
 <a id="nestedatt--services"></a>
 ### Nested Schema for `services`
 
-Optional:
-
-- `monitoring_options` (Attributes) Configuration options for monitoring the service. These options vary depending on whether the monitoring type is internal or third-party. (see [below for nested schema](#nestedatt--services--monitoring_options))
-
 Read-Only:
 
 - `auto_incident` (Boolean) Create an incident automatically when this service is down and close it if/when it comes back up.
@@ -63,12 +59,13 @@ Read-Only:
 - `incoming_webhook_url` (String) This is field is populated from `inbound_email_id`, if the `monitoring` is set to `webhook` and the `webhook_monitoring_service` is set.
 - `inserted_at` (String) Datetime at which the service was inserted.
 - `is_up` (Boolean) Is the monitored service up?
-- `monitoring` (String) Enum: `null` `""` `"internal"` `"3rd_party"` `"webhook"`
+- `monitoring` (String) Enum: `""` `"internal"` `"3rd_party"` `"webhook"`
   Monitoring types:
-  - `null` or `""` - No monitoring.
+  - `""` - No monitoring.
   - `internal` - StatusPal monitoring.
   - `3rd_party` - 3rd Party monitoring.
   - `webhook` - Incoming webhook monitoring.
+- `monitoring_options` (Attributes) Configuration options for monitoring the service. These options vary depending on whether the monitoring type is internal or third-party. (see [below for nested schema](#nestedatt--services--monitoring_options))
 - `name` (String) The name of the service.
 - `order` (Number) Service's position in the service list.
 - `parent_id` (String) The service parent ID.
@@ -109,7 +106,7 @@ Read-Only:
 <a id="nestedatt--services--monitoring_options"></a>
 ### Nested Schema for `services.monitoring_options`
 
-Optional:
+Read-Only:
 
 - `headers` (Attributes List) A list of header objects to be sent with the monitoring request. Each header should include a `key` and `value`. (see [below for nested schema](#nestedatt--services--monitoring_options--headers))
 - `keyword_down` (String) A custom keyword that indicates a 'down' status when monitoring a third-party service. This keyword is used to parse and understand service.
@@ -119,7 +116,7 @@ Optional:
 <a id="nestedatt--services--monitoring_options--headers"></a>
 ### Nested Schema for `services.monitoring_options.headers`
 
-Optional:
+Read-Only:
 
 - `key` (String) The key of the header. Example: `Authorization`.
 - `value` (String) The value of the header. Example: `Bearer token`.

docs/data-sources/status_pages.md

@@ -37,7 +37,7 @@ data "statuspal_status_pages" "all" {
 Read-Only:
 
 - `about` (String) Customize the about information displayed in your status page.
-- `allowed_email_domains` (String) Allowed email domains. Each domain should be separated by \n (e.g., 'acme.corp\nnapster.com')
+- `allowed_email_domains` (String) Users with these domains in their email address will be able to sign up via status page invite link. Each domain should be separated by `\n` (e.g., `acme.corp\nnapster.com`).
 - `bg_image` (String) Background image url of the status page.
 - `calendar_enabled` (Boolean) Allow your customers to receive updates via iCalendar feed.
 - `captcha_enabled` (Boolean) Enable captchas (this option is only available when the status page is member restricted).
@@ -101,7 +101,6 @@ Read-Only:
 - `subscribers_enabled` (Boolean) Allow email customers to receive email notifications.
 - `support_email` (String) Your company's support email.
 - `teams_notifications_enabled` (Boolean) Allow your customers to receive notifications on Microsoft Teams.
-- `theme_configs` (Object) Theme configuration for the status page. (see [below for nested schema](#nestedatt--status_pages--theme_configs))
 - `theme_selected` (String) The selected theme for state page, it can be "default" and "big-logo".
 - `time_format` (String) Display timestamps of incidents and updates in this format.
 - `time_zone` (String) The primary timezone the status page uses to display incidents (e.g. "Europe/Berlin").
@@ -127,22 +126,6 @@ Read-Only:
 - `url` (String) The website to your company, project or service.
 - `zoom_notifications_enabled` (Boolean) Allow your customers to receive notifications on Zoom.
 
-<a id="nestedatt--status_pages--theme_configs"></a>
-### Nested Schema for `status_pages.theme_configs`
-
-Read-Only:
-
-- `header_bg_color1` (String)
-- `header_bg_color2` (String)
-- `header_fg_color` (String)
-- `incident_header_color` (String)
-- `link_color` (String)
-- `status_maintenance_color` (String)
-- `status_major_color` (String)
-- `status_minor_color` (String)
-- `status_ok_color` (String)
-
-
 <a id="nestedatt--status_pages--translations"></a>
 ### Nested Schema for `status_pages.translations`
 

docs/index.md

@@ -18,7 +18,7 @@ terraform {
   required_providers {
     statuspal = {
       source  = "statuspal/statuspal"
-      version = "0.2.11"
+      version = "0.2.12"
     }
   }
 

docs/resources/service.md

@@ -57,9 +57,9 @@ Optional:
 - `description` (String) The description of the service.
 - `display_response_time_chart` (Boolean) Display response time chart?
 - `display_uptime_graph` (Boolean) Display uptime graph?
-- `monitoring` (String) Enum: `null` `""` `"internal"` `"3rd_party"` `"webhook"`
+- `monitoring` (String) Enum: `""` `"internal"` `"3rd_party"` `"webhook"`
   Monitoring types:
-  - `null` or `""` - No monitoring.
+  - `""` - No monitoring.
   - `internal` - StatusPal monitoring.
   - `3rd_party` - 3rd Party monitoring.
   - `webhook` - Incoming webhook monitoring.

docs/resources/status_page.md

@@ -48,7 +48,7 @@ Required:
 Optional:
 
 - `about` (String) Customize the about information displayed in your status page.
-- `allowed_email_domains` (String) Allowed email domains. Each domain should be separated by \n (e.g., 'acme.corp\nnapster.com')
+- `allowed_email_domains` (String) Users with these domains in their email address will be able to sign up via status page invite link. Each domain should be separated by `\n` (e.g., `acme.corp\nnapster.com`).
 - `calendar_enabled` (Boolean) Allow your customers to receive updates via iCalendar feed.
 - `captcha_enabled` (Boolean) Enable captchas (this option is only available when the status page is member restricted).
 - `current_incidents_position` (String) The incident position displayed in the status page, it can be "below_services" and "above_services".
@@ -107,7 +107,6 @@ Optional:
 - `subscribers_enabled` (Boolean) Allow email customers to receive email notifications.
 - `support_email` (String) Your company's support email.
 - `teams_notifications_enabled` (Boolean) Allow your customers to receive notifications on Microsoft Teams.
-- `theme_configs` (Object) Theme configuration for the status page. (see [below for nested schema](#nestedatt--status_page--theme_configs))
 - `theme_selected` (String) The selected theme for state page, it can be "default" and "big-logo".
 - `time_format` (String) Display timestamps of incidents and updates in this format.
 - `translations` (Attributes Map) A translations object. For example:
@@ -128,6 +127,7 @@ Optional:
 - `tweeting_enabled` (Boolean) Allows to send tweets when creating or updating an incident.
 - `twitter_public_screen_name` (String) Twitter handle name (e.g. yourcompany).
 - `uptime_graph_days` (Number) Uptime graph period.
+- `zoom_notifications_enabled` (Boolean) Allow your customers to receive notifications on Zoom.
 
 Read-Only:
 
@@ -136,23 +136,6 @@ Read-Only:
 - `inserted_at` (String) Datetime at which the status page was inserted.
 - `logo` (String) Logo url of the status page.
 - `updated_at` (String) Datetime at which the status page was last updated.
-- `zoom_notifications_enabled` (Boolean) Allow your customers to receive notifications on Zoom.
-
-<a id="nestedatt--status_page--theme_configs"></a>
-### Nested Schema for `status_page.theme_configs`
-
-Required:
-
-- `header_bg_color1` (String)
-- `header_bg_color2` (String)
-- `header_fg_color` (String)
-- `incident_header_color` (String)
-- `link_color` (String)
-- `status_maintenance_color` (String)
-- `status_major_color` (String)
-- `status_minor_color` (String)
-- `status_ok_color` (String)
-
 
 <a id="nestedatt--status_page--translations"></a>
 ### Nested Schema for `status_page.translations`

examples/provider/provider.tf

@@ -3,7 +3,7 @@ terraform {
   required_providers {
     statuspal = {
       source  = "statuspal/statuspal"
-      version = "0.2.11"
+      version = "0.2.12"
     }
   }
 

internal/client/client.go

@@ -26,13 +26,10 @@ func NewClient(api_key *string, region *string, test_url *string) (*Client, erro
 	}
 
 	if *region == "EU" || *region == "US" {
-		var topLevelDomain string
-		switch *region {
-		case "EU":
-			topLevelDomain = "eu"
-		case "US":
-			topLevelDomain = "io"
-		}
+		topLevelDomain := map[string]string{
+			"EU": "eu",
+			"US": "io",
+		}[*region]
 
 		c.HostURL = fmt.Sprintf("https://statuspal.%s/api/v2", topLevelDomain)
 	} else if env == "TEST" {