Skip to content

Conversation

@duedares-rvj
Copy link
Member

@duedares-rvj duedares-rvj commented Dec 5, 2025

🔧 Changes

  • resource/auth0_branding_phone_notification_template: Create and update phone notification templates
  • data-source/auth0_branding_phone_notification_template: Retrieve and manage existing phone notification templates
  • Added relevant test cases and examples

Example:

resource "auth0_branding_phone_notification_template" "otp_enrollment" {
  type     = "otp_enroll"
  disabled = false
  content {
    from = "+1234567890"
    body {
      text  = "Your enrollment code is: @{code}"
      voice = "Your enrollment code is @{code}"
    }
  }
}

data "auth0_branding_phone_notification_template" "otp_enrollment" {
  template_id = "tem_xxxxxxxxxxxxxxxxx"
}

📚 References

https://auth0.com/docs/api/management/v2/branding/create-phone-template

🔬 Testing

Tests added and recorded
make test FILTER=TestAccDataSourcePhoneNotificationTemplate
make test FILTER=TestAccPhoneNotificationTemplate

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@duedares-rvj duedares-rvj requested a review from a team as a code owner December 5, 2025 15:43
@duedares-rvj duedares-rvj self-assigned this Dec 5, 2025
"disabled": {
Type: schema.TypeBool,
Optional: true,
Computed: true,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

disabled
Should it be computed?

Copy link
Contributor

@kushalshit27 kushalshit27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@developerkunal developerkunal enabled auto-merge (squash) December 9, 2025 05:55
@developerkunal developerkunal merged commit 7c2e440 into main Dec 9, 2025
4 checks passed
@developerkunal developerkunal deleted the DXCDT-1227/phone_templates branch December 9, 2025 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants