Add phone notification template support to terraform generation#1514
Open
bkiran6398 wants to merge 2 commits intomainfrom
Open
Add phone notification template support to terraform generation#1514bkiran6398 wants to merge 2 commits intomainfrom
bkiran6398 wants to merge 2 commits intomainfrom
Conversation
- Introduced PhoneNotificationTemplateAPI to manage phone notification templates. - Updated APIV2 to include PhoneNotificationTemplate. - Enhanced resource fetchers to support fetching phone notification templates. - Modified terraform command parsing to accommodate new resource type. - Added tests for phone notification template fetching functionality.
d675301 to
6911a52
Compare
…essed - Added a check to ensure that only templates with a non-empty ID are appended to the import data list in the FetchData method of the phoneNotificationTemplateResourceFetcher. - This change prevents potential errors when processing templates without valid IDs, improving the robustness of the data fetching logic.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔧 Changes
Adds
auth0_phone_notification_templateresource support to theauth0 tf generatecommand.PhoneNotificationTemplateAPIinterface backed by the go-auth0 v2 SDK (m.Branding.Phone.Templates)phoneNotificationTemplateResourceFetcherthat lists phone notification templates and generates Terraform import blocksparseResourceFetchersto accept*auth0.APIV2alongside the existing v1 API, enabling v2 SDK usage in the terraform generation flowauth0_phone_notification_templateto the default resources list so it is included in generation by default📚 References
🔬 Testing
phoneNotificationTemplateResourceFetcher:TestTerraformInputs_ParseResourceFetchersupdated and extended with a new test case for the resource📝 Checklist