Skip to content

Commit

Permalink
Merging dev into main for v1.51.0 (#1320)
Browse files Browse the repository at this point in the history
* Bump hashicorp/setup-terraform from 3.1.0 to 3.1.1 (#1024)

Bumps [hashicorp/setup-terraform](https://github.com/hashicorp/setup-terraform) from 3.1.0 to 3.1.1.
- [Release notes](https://github.com/hashicorp/setup-terraform/releases)
- [Changelog](https://github.com/hashicorp/setup-terraform/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/setup-terraform/compare/v3.1.0...v3.1.1)

---
updated-dependencies:
- dependency-name: hashicorp/setup-terraform
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* bug/devtooling 362 - Evaluation forms incorrectly assigning published attribute (#1022)

* Correctly sets published feature

* Added functionality to set published field correctly

* Fixed docs

* Bug/devtooling 444: Custom resolver function for outbound campaign script_id (#1014)

* Resolving campaign script_id to default outbound script data source when appropriate

* Created constants package

* editing scripts proxy so that the datasource catches Default scripts

* Finished writing tests for new campaign field custom resolver

* Fixed linting errors in campaigns package

* implemented caching for campaigns exporter

* minor changes to a comment

* Add deleteCacheItem func + some refactoring work

* Refactoring call to resolveToDataSource function and added unit test

* Fixing script data source test

* Bump github.com/nyaruka/phonenumbers from 1.3.4 to 1.3.5 (#1026)

Bumps [github.com/nyaruka/phonenumbers](https://github.com/nyaruka/phonenumbers) from 1.3.4 to 1.3.5.
- [Release notes](https://github.com/nyaruka/phonenumbers/releases)
- [Changelog](https://github.com/nyaruka/phonenumbers/blob/main/CHANGELOG.md)
- [Commits](https://github.com/nyaruka/phonenumbers/compare/v1.3.4...v1.3.5)

---
updated-dependencies:
- dependency-name: github.com/nyaruka/phonenumbers
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Devtooling-162: Refactored idp salesforce (#999)

* Refactored idp salesforce

* Fixing docs

* Changed function name

* Removed original file

* Changed Build Function Call to fix type difference (#1029)

* fixing test for published attribute (#1030)

* No jira: fixing up a few acceptance tests (#1031)

* fixed journey segment update

* Adding check to journey segment tests to validate is feature is supported

* preventing panic in export helper function

* Adding sleep to milestone data source test

* Cleaning up fmt.Print statements appearing in export tests

* undoing changes to segment resource file

* Fix routing queue (#1028)

* README update (#1025)

* Pre-verify if the tfState is in sync with the exporter (#1027)

* Add state comapartors

* Upddate review comments

* Devtooling-591: Consistency Checker Refactor (#1032)

* Moved where consistency checker is created in resources

* Cleaned up consitency checker file

* Added resource name to message

* Conditionally adding name

* Made requested changes

* Feature/devtooling 589 1 (#1035)

* update doc

* fix reviews

* Fix acceptance tests - TCA and Prod (#1033)

* Fix edge,export,script test,Jenkin to run tca&prod

* Fix export & integration tests

* Fix auth tests

Fix auth tests

* fix tests

* resolve conflicts

* move assignregion func, remove commented function

modify to return empty string - assignregion func

return default region

---------

Co-authored-by: Monisha Padmavathi Ragavan <[email protected]>

* Bug/devtooling 597 - Routing email route test failing  (#1039)

* Added call to cleanup function

* linter

* adding hotfix to pr labeler config file (#1042)

* bug/DEVTOOLING-436 - Added cleanup function for webdeployments resources (#1038)

* Added cleanup function for webdeployments resources

* fixing conflict

* Made requested changes

* Made requested changes

* Bump github.com/hashicorp/terraform-plugin-sdk/v2 from 2.33.0 to 2.34.0 (#1047)

Bumps [github.com/hashicorp/terraform-plugin-sdk/v2](https://github.com/hashicorp/terraform-plugin-sdk) from 2.33.0 to 2.34.0.
- [Release notes](https://github.com/hashicorp/terraform-plugin-sdk/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-sdk/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/terraform-plugin-sdk/compare/v2.33.0...v2.34.0)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-sdk/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bug/devtooling 287 temp disable cchecker (#1037)

* removed consistency checker

* changed DID number

* phoneUpdated

* Merging dev to main for v1.38.0 (#1036)

* Bump golang.org/x/net from 0.22.0 to 0.23.0 (#951)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.22.0 to 0.23.0.
- [Commits](https://github.com/golang/net/compare/v0.22.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add team to custom resolver (#950)

* Devtooling 432: Refactor Conditional Group Routing Rules into its own resource (#921)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Fixed routing queue tests

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Fixed type

* Changing target branch of dependabot PRs to dev (#956)

* Fix User Resource Panics (#949)

* moving changes (#964)

* Reverting conditional group routing resoure (#965)

* Remove use_supported_content_profile (#962)

* bug/ DEVTOOLING-516: Allow Multiple inputs for BYMONTHDAY and BYMONTH (#953)

* Allow Multiple inputs for BYMONTHDAY and BYMONTH

* Update go.mod

* Added Error Message to Validator

* Bug/devtooling-530: Fixing panic during export caused by MemberGroupsResolver (#957)

* Changing groups.owner_ids to optional and computed

* Preventing panic inside MemberGroupsResolver with some refactoring

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag… (#946)

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag.  This is pulling the index.md out of git hub and overriding what ever is being generated.

* DEVTOOLING-417: Fixed a mispelling in a package alias

* DEVTOOLING-417 Accidently added cache code

* avoiding infinite loop in flow data source read (#966)

* feat/Devtooling-448 oauth cred cache (#945)

* DEVTOOLING-448:  Initial commit of code

* DEVTOOLING-448:  Added caching while creating a Genesys Cloud OAuth Client specific so we dont have to force the user embed a secret

* DEVTOOLING-448: Refactored retrieveCachedOauthClientSecret code

* DEVTOOLING-448: Committing interm work

* DEVTOOLING-448:  Reworked how we handle the addition of roles for user credentials

* DEVTOOLING-448: Removed the ToPtrStr function

* Fixing go.mod and running go generate (#967)

* feat/Devtooling-459: Added BuildAPIDiagnosticError function calls (#959)

* added buildDiag function p1

* Adding call to BuildApiDiag

* BuildAPIDiag functions added

* Added util.BuildAPIDiag function to refactored resources

* Added call to BuildAPIDiag in resources

* Update go.mod

* Fixed tests

* Update go.mod

* Added err

* Added err

* Update go.mod

* Requested Changes

* Fix for BotFlows dependencies (#961)

* feat/Devtooling-531: Add proxy and cache to routing queue (#958)

* Added cache to routing queue

* Added proxy and caching to routing queue

* Removed log

* Add consistency checker back

* Fix group cache

* Fix Export cyclic deps (#968)

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Feat/devtooling-177: Refactored orgAuthorization_pairing  (#975)

* refactored orgauthorization_pairing

* Added exporter for response management response asset

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Fixed test

* using resource mutexes in init_test files & linting errors (#973)

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Feature/devtooling 63 (#978)

* Fix exports

* Feat/add flow log level resource (#948)

* add code changes

* add flow log level changes

* testing commit

* getting there, have a create test together

* still trying to get a passing test

* tests have passed! but getting the plan was not empty errors now

* flow log level test

* finally got tests passing

* remove debug logging

* rerun go generate

* rerun go generate

* apply suggestions from PR review

* address PR review

* remove characteristics

* address PR comments

* return total flow log levels when no more are found

---------

Co-authored-by: dginty4 <[email protected]>

* NO-JIRA: Small Change to flow log level (#984)

* Small Change to flow log level

* Fixing typo

* Merge hotfix changes on main into dev (#983)

* moving changes (#960)

* Revert "moving changes (#960)" (#963)

This reverts commit 70d8a8036c52ae2f91aee081fe0d1c4c5451a58f.

* Release Merge to Main (#977)

* Bump golang.org/x/net from 0.22.0 to 0.23.0 (#951)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.22.0 to 0.23.0.
- [Commits](https://github.com/golang/net/compare/v0.22.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add team to custom resolver (#950)

* Devtooling 432: Refactor Conditional Group Routing Rules into its own resource (#921)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Fixed routing queue tests

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Fixed type

* Changing target branch of dependabot PRs to dev (#956)

* Fix User Resource Panics (#949)

* moving changes (#964)

* Reverting conditional group routing resoure (#965)

* Remove use_supported_content_profile (#962)

* bug/ DEVTOOLING-516: Allow Multiple inputs for BYMONTHDAY and BYMONTH (#953)

* Allow Multiple inputs for BYMONTHDAY and BYMONTH

* Update go.mod

* Added Error Message to Validator

* Bug/devtooling-530: Fixing panic during export caused by MemberGroupsResolver (#957)

* Changing groups.owner_ids to optional and computed

* Preventing panic inside MemberGroupsResolver with some refactoring

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag… (#946)

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag.  This is pulling the index.md out of git hub and overriding what ever is being generated.

* DEVTOOLING-417: Fixed a mispelling in a package alias

* DEVTOOLING-417 Accidently added cache code

* avoiding infinite loop in flow data source read (#966)

* feat/Devtooling-448 oauth cred cache (#945)

* DEVTOOLING-448:  Initial commit of code

* DEVTOOLING-448:  Added caching while creating a Genesys Cloud OAuth Client specific so we dont have to force the user embed a secret

* DEVTOOLING-448: Refactored retrieveCachedOauthClientSecret code

* DEVTOOLING-448: Committing interm work

* DEVTOOLING-448:  Reworked how we handle the addition of roles for user credentials

* DEVTOOLING-448: Removed the ToPtrStr function

* Fixing go.mod and running go generate (#967)

* feat/Devtooling-459: Added BuildAPIDiagnosticError function calls (#959)

* added buildDiag function p1

* Adding call to BuildApiDiag

* BuildAPIDiag functions added

* Added util.BuildAPIDiag function to refactored resources

* Added call to BuildAPIDiag in resources

* Update go.mod

* Fixed tests

* Update go.mod

* Added err

* Added err

* Update go.mod

* Requested Changes

* Fix for BotFlows dependencies (#961)

* feat/Devtooling-531: Add proxy and cache to routing queue (#958)

* Added cache to routing queue

* Added proxy and caching to routing queue

* Removed log

* Add consistency checker back

* Fix group cache

* Fix Export cyclic deps (#968)

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Feat/devtooling-177: Refactored orgAuthorization_pairing  (#975)

* refactored orgauthorization_pairing

* Added exporter for response management response asset

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Fixed test

* using resource mutexes in init_test files & linting errors (#973)

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Feature/devtooling 63 (#978)

* Fix exports

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Charlie Conneely <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Fix Exporter for non-flow resources (#982)

* Fix Exporter for non-flow resources

* Added check (#981)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: HemanthDogiparthi12 <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Bump github.com/hashicorp/terraform-plugin-docs from 0.18.0 to 0.19.1 (#992)

* Bump github.com/hashicorp/terraform-plugin-docs from 0.18.0 to 0.19.1

Bumps [github.com/hashicorp/terraform-plugin-docs](https://github.com/hashicorp/terraform-plugin-docs) from 0.18.0 to 0.19.1.
- [Release notes](https://github.com/hashicorp/terraform-plugin-docs/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-docs/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/terraform-plugin-docs/compare/v0.18.0...v0.19.1)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-docs
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* rerunning go generate after tf-docs package upgrade

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Charlie Conneely <[email protected]>

* Devtooling 432: Conditonal Group Routing (#970)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Added CGR and toggle

* Fixed some errors

* Added queue proxy to CGR

* Added toggle to get all

* Made some changes

* Added CGR toggle package

* Moved CGR toggle package

* Ran linter

* Checking queue id in tests

* Small change

* bug/Devtooling-458: media retention policy creation failing with error on time interval (#980)

* added checks

* Checks for time interval attributes

* Fixing TimeInterval

* Added BuildFunc for timeInterval

* Added returned errors for invalid time Interval

* Fixed buildFunctions

* Generate Docs

* Fixed exporter

* Getting rid of go vet warning about copying a lock value to a variable (#991)

* No jira: util function for getting cursor value from next uri  (#990)

* moving changes (#960)

* Revert "moving changes (#960)" (#963)

This reverts commit 70d8a8036c52ae2f91aee081fe0d1c4c5451a58f.

* Release Merge to Main (#977)

* Bump golang.org/x/net from 0.22.0 to 0.23.0 (#951)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.22.0 to 0.23.0.
- [Commits](https://github.com/golang/net/compare/v0.22.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add team to custom resolver (#950)

* Devtooling 432: Refactor Conditional Group Routing Rules into its own resource (#921)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Fixed routing queue tests

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Fixed type

* Changing target branch of dependabot PRs to dev (#956)

* Fix User Resource Panics (#949)

* moving changes (#964)

* Reverting conditional group routing resoure (#965)

* Remove use_supported_content_profile (#962)

* bug/ DEVTOOLING-516: Allow Multiple inputs for BYMONTHDAY and BYMONTH (#953)

* Allow Multiple inputs for BYMONTHDAY and BYMONTH

* Update go.mod

* Added Error Message to Validator

* Bug/devtooling-530: Fixing panic during export caused by MemberGroupsResolver (#957)

* Changing groups.owner_ids to optional and computed

* Preventing panic inside MemberGroupsResolver with some refactoring

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag… (#946)

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag.  This is pulling the index.md out of git hub and overriding what ever is being generated.

* DEVTOOLING-417: Fixed a mispelling in a package alias

* DEVTOOLING-417 Accidently added cache code

* avoiding infinite loop in flow data source read (#966)

* feat/Devtooling-448 oauth cred cache (#945)

* DEVTOOLING-448:  Initial commit of code

* DEVTOOLING-448:  Added caching while creating a Genesys Cloud OAuth Client specific so we dont have to force the user embed a secret

* DEVTOOLING-448: Refactored retrieveCachedOauthClientSecret code

* DEVTOOLING-448: Committing interm work

* DEVTOOLING-448:  Reworked how we handle the addition of roles for user credentials

* DEVTOOLING-448: Removed the ToPtrStr function

* Fixing go.mod and running go generate (#967)

* feat/Devtooling-459: Added BuildAPIDiagnosticError function calls (#959)

* added buildDiag function p1

* Adding call to BuildApiDiag

* BuildAPIDiag functions added

* Added util.BuildAPIDiag function to refactored resources

* Added call to BuildAPIDiag in resources

* Update go.mod

* Fixed tests

* Update go.mod

* Added err

* Added err

* Update go.mod

* Requested Changes

* Fix for BotFlows dependencies (#961)

* feat/Devtooling-531: Add proxy and cache to routing queue (#958)

* Added cache to routing queue

* Added proxy and caching to routing queue

* Removed log

* Add consistency checker back

* Fix group cache

* Fix Export cyclic deps (#968)

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Feat/devtooling-177: Refactored orgAuthorization_pairing  (#975)

* refactored orgauthorization_pairing

* Added exporter for response management response asset

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Fixed test

* using resource mutexes in init_test files & linting errors (#973)

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Feature/devtooling 63 (#978)

* Fix exports

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Charlie Conneely <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Fix Exporter for non-flow resources (#982)

* Fix Exporter for non-flow resources

* Added check (#981)

* Made function for parsing cursor from next uri

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: HemanthDogiparthi12 <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Fix acceptance tests (#979)

* fix acceptance test

fix test

fix test

fix test

fix test

fix tests

fix tests

fix tests

fix tests

fix tests

fix test

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix routing test

fix routing test

fix routing test

fix routing test

fix routing test

fix routing test

fix routing test

fix routing test

fix tests

fix tests

restore files

fix team test

* fix comments

* fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

* fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

* fix tests

fix tests

fix tests

* fix tests

* fix conditional routing test

fix conditional routing test

---------

Co-authored-by: Monisha Padmavathi Ragavan <[email protected]>

* Bump hashicorp/setup-terraform from 3.0.0 to 3.1.0 (#995)

Bumps [hashicorp/setup-terraform](https://github.com/hashicorp/setup-terraform) from 3.0.0 to 3.1.0.
- [Release notes](https://github.com/hashicorp/setup-terraform/releases)
- [Changelog](https://github.com/hashicorp/setup-terraform/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/setup-terraform/compare/v3.0.0...v3.1.0)

---
updated-dependencies:
- dependency-name: hashicorp/setup-terraform
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Improvise exclude attributes to Regex (#988)

* Fix Media call back Read Attributes (#987)

* DEVTOOLING-445: User Roles not returning duplicate roles (#989)

* NO-JIRA-Upgrading-Go-SDK

* Fixed errors

* Fixed conditional group routing

* Fixed some tests

* feat/Devtooling 496: Routing queue Outbound email address independent resource (#954)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Removed accidental commits

* Rebased and fixed errors

* Added evaluated_queue_id

* Fixed some things

* Added CGR and toggle

* Made some changes

* Added CGR toggle package

* Moved CGR toggle package

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Routing queue outbound email address package

* Made changes

* Added tests

* Added more to test

* Fixed test

* Finished resource

* Finished resource

* Made some requested changes

* Added feature toggle stuff

* Added feature toggle

* Made changes

* Fixing change in queue exporter

---------

Co-authored-by: HemanthDogiparthi12 <[email protected]>

* bug/devtooling-498: [CX as Code] genesyscloud_outbound_settings export contains only null data (#996)

* Fixing null export

* Fixing Test

* Added check for export

* Added attribute and fixed test

* Removed Computed attribute

* Added check for reschedule_time_zone_skipped_contacts in test

* Update Export Guide (#971)

* update export uide

* update tf_export.md and resource.tf

* cross check resource.tf and schema, add missing attributes

* update

* update export.md.tmp

* Removed duplicate paragraph and ran go generate

---------

Co-authored-by: Charlie Conneely <[email protected]>

* NO-JIRA: Fixing dev (#1000)

* Fixing dev

* Fixes some errors

* Fixing flow outcome description (#998)

* Devtooling 561 - Members not being added correctly to teams (#1001)

* Fixed bug

* Fixed problem deleting members

* Removed print

* Fix Version Problem in WebDeployments (#997)

* Fix Version Problem in WebDeployments

* Add test cases

* Fix review comments

* no jira: Fixing merge conflicts on dev branch  (#1009)

* moving changes (#960)

* Revert "moving changes (#960)" (#963)

This reverts commit 70d8a8036c52ae2f91aee081fe0d1c4c5451a58f.

* Release Merge to Main (#977)

* Bump golang.org/x/net from 0.22.0 to 0.23.0 (#951)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.22.0 to 0.23.0.
- [Commits](https://github.com/golang/net/compare/v0.22.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add team to custom resolver (#950)

* Devtooling 432: Refactor Conditional Group Routing Rules into its own resource (#921)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Fixed routing queue tests

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Fixed type

* Changing target branch of dependabot PRs to dev (#956)

* Fix User Resource Panics (#949)

* moving changes (#964)

* Reverting conditional group routing resoure (#965)

* Remove use_supported_content_profile (#962)

* bug/ DEVTOOLING-516: Allow Multiple inputs for BYMONTHDAY and BYMONTH (#953)

* Allow Multiple inputs for BYMONTHDAY and BYMONTH

* Update go.mod

* Added Error Message to Validator

* Bug/devtooling-530: Fixing panic during export caused by MemberGroupsResolver (#957)

* Changing groups.owner_ids to optional and computed

* Preventing panic inside MemberGroupsResolver with some refactoring

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag… (#946)

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag.  This is pulling the index.md out of git hub and overriding what ever is being generated.

* DEVTOOLING-417: Fixed a mispelling in a package alias

* DEVTOOLING-417 Accidently added cache code

* avoiding infinite loop in flow data source read (#966)

* feat/Devtooling-448 oauth cred cache (#945)

* DEVTOOLING-448:  Initial commit of code

* DEVTOOLING-448:  Added caching while creating a Genesys Cloud OAuth Client specific so we dont have to force the user embed a secret

* DEVTOOLING-448: Refactored retrieveCachedOauthClientSecret code

* DEVTOOLING-448: Committing interm work

* DEVTOOLING-448:  Reworked how we handle the addition of roles for user credentials

* DEVTOOLING-448: Removed the ToPtrStr function

* Fixing go.mod and running go generate (#967)

* feat/Devtooling-459: Added BuildAPIDiagnosticError function calls (#959)

* added buildDiag function p1

* Adding call to BuildApiDiag

* BuildAPIDiag functions added

* Added util.BuildAPIDiag function to refactored resources

* Added call to BuildAPIDiag in resources

* Update go.mod

* Fixed tests

* Update go.mod

* Added err

* Added err

* Update go.mod

* Requested Changes

* Fix for BotFlows dependencies (#961)

* feat/Devtooling-531: Add proxy and cache to routing queue (#958)

* Added cache to routing queue

* Added proxy and caching to routing queue

* Removed log

* Add consistency checker back

* Fix group cache

* Fix Export cyclic deps (#968)

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Feat/devtooling-177: Refactored orgAuthorization_pairing  (#975)

* refactored orgauthorization_pairing

* Added exporter for response management response asset

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Fixed test

* using resource mutexes in init_test files & linting errors (#973)

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Feature/devtooling 63 (#978)

* Fix exports

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Charlie Conneely <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Fix Exporter for non-flow resources (#982)

* Fix Exporter for non-flow resources

* Added check (#981)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: HemanthDogiparthi12 <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Bump github.com/hashicorp/terraform-plugin-docs from 0.19.1 to 0.19.2 (#1008)

* Bump github.com/hashicorp/terraform-plugin-docs from 0.19.1 to 0.19.2

Bumps [github.com/hashicorp/terraform-plugin-docs](https://github.com/hashicorp/terraform-plugin-docs) from 0.19.1 to 0.19.2.
- [Release notes](https://github.com/hashicorp/terraform-plugin-docs/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-docs/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/terraform-plugin-docs/compare/v0.19.1...v0.19.2)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-docs
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

* ran go generate

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Charlie Conneely <[email protected]>

* DEVTOOLING-352 - provider oauth credential rate limit exceeded (#1004)

* Testing something

* Adding retry

* Fixed retry

* Moved functionality to retries package

* Fixed inport cycle

* Extending sleep

* Added retry to AuthorizeSdk

* Using with retries

* Remving old function

* Feat/devtooling 461 - WithRetries errors include improved logging (#1012)

* calling diag func

* Changed retryable functions to call Diagnostic function

* Converted all withRetries errors to have improved wrapper

* Made requested Changes

* fixing unit test

* Bump golang.org/x/net from 0.23.0 to 0.25.0 (#1017)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.23.0 to 0.25.0.
- [Commits](https://github.com/golang/net/compare/v0.23.0...v0.25.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix for issue 993 : Changed message_column to optional and add verification logic (#1013)

* Changed message_column to optional and add verification logic

* applied safer nil value handling

* go generate (#1018)

* bug/devtooling 560 - Nil error in telephony_providers_edges_phone (#1015)

* Removed Print Statement that throws Nil error and added random id to test to prevent failing

* golint changes made

* Revert "bug/devtooling 560 - Nil error in telephony_providers_edges_phone (#1…" (#1019)

This reverts commit 3ff66d472071905b18b4f66f6ee4b3d3725600e7.

* bug/devtooling 560 - CX as Code crash when creating phones (#1020)

* Removed Print Statement that throws Nil error and added random id to test to prevent failing

* golint changes made

* Made requested Change

* feat/devtooling 579 - Improved Logging for Non-API errors (#1021)

* Adding improved non-api diag errors

* Pulling Merged PR

* Added improved error

* Added more logging

* Ran go vet

* Updated docs

* Bump hashicorp/setup-terraform from 3.1.0 to 3.1.1 (#1024)

Bumps [hashicorp/setup-terraform](https://github.com/hashicorp/setup-terraform) from 3.1.0 to 3.1.1.
- [Release notes](https://github.com/hashicorp/setup-terraform/releases)
- [Changelog](https://github.com/hashicorp/setup-terraform/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/setup-terraform/compare/v3.1.0...v3.1.1)

---
updated-dependencies:
- dependency-name: hashicorp/setup-terraform
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* bug/devtooling 362 - Evaluation forms incorrectly assigning published attribute (#1022)

* Correctly sets published feature

* Added functionality to set published field correctly

* Fixed docs

* Bug/devtooling 444: Custom resolver function for outbound campaign script_id (#1014)

* Resolving campaign script_id to default outbound script data source when appropriate

* Created constants package

* editing scripts proxy so that the datasource catches Default scripts

* Finished writing tests for new campaign field custom resolver

* Fixed linting errors in campaigns package

* implemented caching for campaigns exporter

* minor changes to a comment

* Add deleteCacheItem func + some refactoring work

* Refactoring call to resolveToDataSource function and added unit test

* Fixing script data source test

* Bump github.com/nyaruka/phonenumbers from 1.3.4 to 1.3.5 (#1026)

Bumps [github.com/nyaruka/phonenumbers](https://github.com/nyaruka/phonenumbers) from 1.3.4 to 1.3.5.
- [Release notes](https://github.com/nyaruka/phonenumbers/releases)
- [Changelog](https://github.com/nyaruka/phonenumbers/blob/main/CHANGELOG.md)
- [Commits](https://github.com/nyaruka/phonenumbers/compare/v1.3.4...v1.3.5)

---
updated-dependencies:
- dependency-name: github.com/nyaruka/phonenumbers
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Devtooling-162: Refactored idp salesforce (#999)

* Refactored idp salesforce

* Fixing docs

* Changed function name

* Removed original file

* Changed Build Function Call to fix type difference (#1029)

* fixing test for published attribute (#1030)

* No jira: fixing up a few acceptance tests (#1031)

* fixed journey segment update

* Adding check to journey segment tests to validate is feature is supported

* preventing panic in export helper function

* Adding sleep to milestone data source test

* Cleaning up fmt.Print statements appearing in export tests

* undoing changes to segment resource file

* Fix routing queue (#1028)

* README update (#1025)

* Pre-verify if the tfState is in sync with the exporter (#1027)

* Add state comapartors

* Upddate review comments

* Devtooling-591: Consistency Checker Refactor (#1032)

* Moved where consistency checker is created in resources

* Cleaned up consitency checker file

* Added resource name to message

* Conditionally adding name

* Made requested changes

* Feature/devtooling 589 1 (#1035)

* update doc

* fix reviews

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: HemanthDogiparthi12 <[email protected]>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>
Co-authored-by: ben-howarth-genesys <[email protected]>
Co-authored-by: monishapadmavathi <[email protected]>
Co-authored-by: Monisha Padmavathi Ragavan <[email protected]>
Co-authored-by: jenissabarrera <[email protected]>
Co-authored-by: Taisuk Choi <[email protected]>

* NO-JIRA: Fixing nil pointer in consistency checker (#1041)

* Fixed nil pointer in consistency checker

* Fixed a few more nil pointers

* Fixing nil pointer (#1045)

* merged with dev

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Charlie Conneely <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: HemanthDogiparthi12 <[email protected]>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>
Co-authored-by: ben-howarth-genesys <[email protected]>
Co-authored-by: monishapadmavathi <[email protected]>
Co-authored-by: Monisha Padmavathi Ragavan <[email protected]>
Co-authored-by: jenissabarrera <[email protected]>
Co-authored-by: Taisuk Choi <[email protected]>

* Fixing up provider docs (#1046)

* Fix Export Version problem (#1049)

* Logging error in queue create/update instead of failing (#1050)

* Refactor/devtooling 395 (#1044)

* refactor/DEVTOOLING-395 - adding architect_schedules package

* Addressed PR comments

* added caching in schedules

* Report Generation & testcase fix (#1051)

* Report generation

generate report

generate report

* fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

* fix tests

* fix tests

* fix tests

fix tests

fix tests

* fix tests

* fix report

* fix report

* fix tests

fix tests

* Bump github.com/hashicorp/terraform-plugin-sdk/v2 from 2.33.0 to 2.34.0 (#1047)

Bumps [github.com/hashicorp/terraform-plugin-sdk/v2](https://github.com/hashicorp/terraform-plugin-sdk) from 2.33.0 to 2.34.0.
- [Release notes](https://github.com/hashicorp/terraform-plugin-sdk/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-sdk/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/terraform-plugin-sdk/compare/v2.33.0...v2.34.0)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-sdk/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bug/devtooling 287 temp disable cchecker (#1037)

* removed consistency checker

* changed DID number

* phoneUpdated

* Merging dev to main for v1.38.0 (#1036)

* Bump golang.org/x/net from 0.22.0 to 0.23.0 (#951)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.22.0 to 0.23.0.
- [Commits](https://github.com/golang/net/compare/v0.22.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add team to custom resolver (#950)

* Devtooling 432: Refactor Conditional Group Routing Rules into its own resource (#921)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Fixed routing queue tests

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Fixed type

* Changing target branch of dependabot PRs to dev (#956)

* Fix User Resource Panics (#949)

* moving changes (#964)

* Reverting conditional group routing resoure (#965)

* Remove use_supported_content_profile (#962)

* bug/ DEVTOOLING-516: Allow Multiple inputs for BYMONTHDAY and BYMONTH (#953)

* Allow Multiple inputs for BYMONTHDAY and BYMONTH

* Update go.mod

* Added Error Message to Validator

* Bug/devtooling-530: Fixing panic during export caused by MemberGroupsResolver (#957)

* Changing groups.owner_ids to optional and computed

* Preventing panic inside MemberGroupsResolver with some refactoring

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag… (#946)

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag.  This is pulling the index.md out of git hub and overriding what ever is being generated.

* DEVTOOLING-417: Fixed a mispelling in a package alias

* DEVTOOLING-417 Accidently added cache code

* avoiding infinite loop in flow data source read (#966)

* feat/Devtooling-448 oauth cred cache (#945)

* DEVTOOLING-448:  Initial commit of code

* DEVTOOLING-448:  Added caching while creating a Genesys Cloud OAuth Client specific so we dont have to force the user embed a secret

* DEVTOOLING-448: Refactored retrieveCachedOauthClientSecret code

* DEVTOOLING-448: Committing interm work

* DEVTOOLING-448:  Reworked how we handle the addition of roles for user credentials

* DEVTOOLING-448: Removed the ToPtrStr function

* Fixing go.mod and running go generate (#967)

* feat/Devtooling-459: Added BuildAPIDiagnosticError function calls (#959)

* added buildDiag function p1

* Adding call to BuildApiDiag

* BuildAPIDiag functions added

* Added util.BuildAPIDiag function to refactored resources

* Added call to BuildAPIDiag in resources

* Update go.mod

* Fixed tests

* Update go.mod

* Added err

* Added err

* Update go.mod

* Requested Changes

* Fix for BotFlows dependencies (#961)

* feat/Devtooling-531: Add proxy and cache to routing queue (#958)

* Added cache to routing queue

* Added proxy and caching to routing queue

* Removed log

* Add consistency checker back

* Fix group cache

* Fix Export cyclic deps (#968)

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Feat/devtooling-177: Refactored orgAuthorization_pairing  (#975)

* refactored orgauthorization_pairing

* Added exporter for response management response asset

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Fixed test

* using resource mutexes in init_test files & linting errors (#973)

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Feature/devtooling 63 (#978)

* Fix exports

* Feat/add flow log level resource (#948)

* add code changes

* add flow log level changes

* testing commit

* getting there, have a create test together

* still trying to get a passing test

* tests have passed! but getting the plan was not empty errors now

* flow log level test

* finally got tests passing

* remove debug logging

* rerun go generate

* rerun go generate

* apply suggestions from PR review

* address PR review

* remove characteristics

* address PR comments

* return total flow log levels when no more are found

---------

Co-authored-by: dginty4 <[email protected]>

* NO-JIRA: Small Change to flow log level (#984)

* Small Change to flow log level

* Fixing typo

* Merge hotfix changes on main into dev (#983)

* moving changes (#960)

* Revert "moving changes (#960)" (#963)

This reverts commit 70d8a8036c52ae2f91aee081fe0d1c4c5451a58f.

* Release Merge to Main (#977)

* Bump golang.org/x/net from 0.22.0 to 0.23.0 (#951)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.22.0 to 0.23.0.
- [Commits](https://github.com/golang/net/compare/v0.22.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add team to custom resolver (#950)

* Devtooling 432: Refactor Conditional Group Routing Rules into its own resource (#921)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Fixed routing queue tests

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Fixed type

* Changing target branch of dependabot PRs to dev (#956)

* Fix User Resource Panics (#949)

* moving changes (#964)

* Reverting conditional group routing resoure (#965)

* Remove use_supported_content_profile (#962)

* bug/ DEVTOOLING-516: Allow Multiple inputs for BYMONTHDAY and BYMONTH (#953)

* Allow Multiple inputs for BYMONTHDAY and BYMONTH

* Update go.mod

* Added Error Message to Validator

* Bug/devtooling-530: Fixing panic during export caused by MemberGroupsResolver (#957)

* Changing groups.owner_ids to optional and computed

* Preventing panic inside MemberGroupsResolver with some refactoring

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag… (#946)

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag.  This is pulling the index.md out of git hub and overriding what ever is being generated.

* DEVTOOLING-417: Fixed a mispelling in a package alias

* DEVTOOLING-417 Accidently added cache code

* avoiding infinite loop in flow data source read (#966)

* feat/Devtooling-448 oauth cred cache (#945)

* DEVTOOLING-448:  Initial commit of code

* DEVTOOLING-448:  Added caching while creating a Genesys Cloud OAuth Client specific so we dont have to force the user embed a secret

* DEVTOOLING-448: Refactored retrieveCachedOauthClientSecret code

* DEVTOOLING-448: Committing interm work

* DEVTOOLING-448:  Reworked how we handle the addition of roles for user credentials

* DEVTOOLING-448: Removed the ToPtrStr function

* Fixing go.mod and running go generate (#967)

* feat/Devtooling-459: Added BuildAPIDiagnosticError function calls (#959)

* added buildDiag function p1

* Adding call to BuildApiDiag

* BuildAPIDiag functions added

* Added util.BuildAPIDiag function to refactored resources

* Added call to BuildAPIDiag in resources

* Update go.mod

* Fixed tests

* Update go.mod

* Added err

* Added err

* Update go.mod

* Requested Changes

* Fix for BotFlows dependencies (#961)

* feat/Devtooling-531: Add proxy and cache to routing queue (#958)

* Added cache to routing queue

* Added proxy and caching to routing queue

* Removed log

* Add consistency checker back

* Fix group cache

* Fix Export cyclic deps (#968)

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Feat/devtooling-177: Refactored orgAuthorization_pairing  (#975)

* refactored orgauthorization_pairing

* Added exporter for response management response asset

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Fixed test

* using resource mutexes in init_test files & linting errors (#973)

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Feature/devtooling 63 (#978)

* Fix exports

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Charlie Conneely <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Fix Exporter for non-flow resources (#982)

* Fix Exporter for non-flow resources

* Added check (#981)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: HemanthDogiparthi12 <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Bump github.com/hashicorp/terraform-plugin-docs from 0.18.0 to 0.19.1 (#992)

* Bump github.com/hashicorp/terraform-plugin-docs from 0.18.0 to 0.19.1

Bumps [github.com/hashicorp/terraform-plugin-docs](https://github.com/hashicorp/terraform-plugin-docs) from 0.18.0 to 0.19.1.
- [Release notes](https://github.com/hashicorp/terraform-plugin-docs/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-docs/blob/main/CHANGELOG.md)
- [Commits](https://github.com/hashicorp/terraform-plugin-docs/compare/v0.18.0...v0.19.1)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-docs
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* rerunning go generate after tf-docs package upgrade

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Charlie Conneely <[email protected]>

* Devtooling 432: Conditonal Group Routing (#970)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Added CGR and toggle

* Fixed some errors

* Added queue proxy to CGR

* Added toggle to get all

* Made some changes

* Added CGR toggle package

* Moved CGR toggle package

* Ran linter

* Checking queue id in tests

* Small change

* bug/Devtooling-458: media retention policy creation failing with error on time interval (#980)

* added checks

* Checks for time interval attributes

* Fixing TimeInterval

* Added BuildFunc for timeInterval

* Added returned errors for invalid time Interval

* Fixed buildFunctions

* Generate Docs

* Fixed exporter

* Getting rid of go vet warning about copying a lock value to a variable (#991)

* No jira: util function for getting cursor value from next uri  (#990)

* moving changes (#960)

* Revert "moving changes (#960)" (#963)

This reverts commit 70d8a8036c52ae2f91aee081fe0d1c4c5451a58f.

* Release Merge to Main (#977)

* Bump golang.org/x/net from 0.22.0 to 0.23.0 (#951)

Bumps [golang.org/x/net](https://github.com/golang/net) from 0.22.0 to 0.23.0.
- [Commits](https://github.com/golang/net/compare/v0.22.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add team to custom resolver (#950)

* Devtooling 432: Refactor Conditional Group Routing Rules into its own resource (#921)

* Added routing_queue_conditional_group_routing resource

* Rebased

* Fixed routing queue tests

* Removed accidental commits

* Rebased and fixed errors

* Made some requested changes

* Added evaluated_queue_id

* Added warning

* Fixed some things

* Fixed test

* Fixing exporter

* Fixed type

* Changing target branch of dependabot PRs to dev (#956)

* Fix User Resource Panics (#949)

* moving changes (#964)

* Reverting conditional group routing resoure (#965)

* Remove use_supported_content_profile (#962)

* bug/ DEVTOOLING-516: Allow Multiple inputs for BYMONTHDAY and BYMONTH (#953)

* Allow Multiple inputs for BYMONTHDAY and BYMONTH

* Update go.mod

* Added Error Message to Validator

* Bug/devtooling-530: Fixing panic during export caused by MemberGroupsResolver (#957)

* Changing groups.owner_ids to optional and computed

* Preventing panic inside MemberGroupsResolver with some refactoring

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag… (#946)

* DEVTOOLING-417: Removed the go:generate git restore docs/index.md tag.  This is pulling the index.md out of git hub and overriding what ever is being generated.

* DEVTOOLING-417: Fixed a mispelling in a package alias

* DEVTOOLING-417 Accidently added cache code

* avoiding infinite loop in flow data source read (#966)

* feat/Devtooling-448 oauth cred cache (#945)

* DEVTOOLING-448:  Initial commit of code

* DEVTOOLING-448:  Added caching while creating a Genesys Cloud OAuth Client specific so we dont have to force the user embed a secret

* DEVTOOLING-448: Refactored retrieveCachedOauthClientSecret code

* DEVTOOLING-448: Committing interm work

* DEVTOOLING-448:  Reworked how we handle the addition of roles for user credentials

* DEVTOOLING-448: Removed the ToPtrStr function

* Fixing go.mod and running go generate (#967)

* feat/Devtooling-459: Added BuildAPIDiagnosticError function calls (#959)

* added buildDiag function p1

* Adding call to BuildApiDiag

* BuildAPIDiag functions added

* Added util.BuildAPIDiag function to refactored resources

* Added call to BuildAPIDiag in resources

* Update go.mod

* Fixed tests

* Update go.mod

* Added err

* Added err

* Update go.mod

* Requested Changes

* Fix for BotFlows dependencies (#961)

* feat/Devtooling-531: Add proxy and cache to routing queue (#958)

* Added cache to routing queue

* Added proxy and caching to routing queue

* Removed log

* Add consistency checker back

* Fix group cache

* Fix Export cyclic deps (#968)

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Feat/devtooling-177: Refactored orgAuthorization_pairing  (#975)

* refactored orgauthorization_pairing

* Added exporter for response management response asset

* Added refactor label to pr-labeler-config

* Feat/devtooling-499: Added getAll Functionality to ResponseManagement_ResponseAsset (#969)

* adding getALl functionality

* adding exporter

* adding export

* Added getAll functionality

* Update go.mod

* Added Caching

* Added file_content_hash

* Fixed test

* using resource mutexes in init_test files & linting errors (#973)

* Feature/devtooling 63 (#974)

* Introduce new Elements

* Update Attribs

* Introduce scoring Method

* Upgrade ssdk

* remove unwanted logs

* Feature/devtooling 63 (#978)

* Fix exports

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Charlie Conneely <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Fix Exporter for non-flow resources (#982)

* Fix Exporter for non-flow resources

* Added check (#981)

* Made function for parsing cursor from next uri

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: kavinbalagen <[email protected]>
Co-authored-by: HemanthDogiparthi12 <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dginty4 <[email protected]>
Co-authored-by: Brian Moyles <[email protected]>
Co-authored-by: carnellj-genesys <[email protected]>
Co-authored-by: BrianMoyles <[email protected]>

* Fix acceptance tests (#979)

* fix acceptance test

fix test

fix test

fix test

fix test

fix tests

fix tests

fix tests

fix tests

fix tests

fix test

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

fix tests

…
  • Loading branch information
1 parent 58e2ef5 commit 51bf7f5
Show file tree
Hide file tree
Showing 15 changed files with 189 additions and 43 deletions.
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@

# Assigning Declan, Hemanth, and myself as code owners
* @dginty4 @carnellj-genesys @HemanthDogiparthi12 @charliecon @kavinbalagen @BrianMoyles
* @dginty4 @carnellj-genesys @HemanthDogiparthi12 @charliecon @kavinbalagen @BrianMoyles @sureshperiyappan
2 changes: 1 addition & 1 deletion docs/resources/oauth_client.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ resource "genesyscloud_oauth_client" "example-client" {

### Required

- `authorized_grant_type` (String) The OAuth Grant/Client type supported by this client (CODE | TOKEN | SAML2BEARER | PASSWORD | CLIENT-CREDENTIALS).
- `authorized_grant_type` (String) The OAuth Grant/Client type supported by this client (CODE | TOKEN | SAML2-BEARER | PASSWORD | CLIENT-CREDENTIALS).
- `name` (String) The name of the OAuth client.

### Optional
Expand Down
1 change: 1 addition & 0 deletions docs/resources/responsemanagement_response.md
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ Required:
Optional:

- `content_type` (String) Response text content type.
- `type` (String) Response text type.


<a id="nestedblock--footer"></a>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
package oauth_client

import (
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
"terraform-provider-genesyscloud/genesyscloud/provider"
resourceExporter "terraform-provider-genesyscloud/genesyscloud/resource_exporter"
registrar "terraform-provider-genesyscloud/genesyscloud/resource_register"

"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
)

const (
Expand Down Expand Up @@ -73,10 +74,10 @@ func ResourceOAuthClient() *schema.Resource {
Optional: true,
},
"authorized_grant_type": {
Description: "The OAuth Grant/Client type supported by this client (CODE | TOKEN | SAML2BEARER | PASSWORD | CLIENT-CREDENTIALS).",
Description: "The OAuth Grant/Client type supported by this client (CODE | TOKEN | SAML2-BEARER | PASSWORD | CLIENT-CREDENTIALS).",
Type: schema.TypeString,
Required: true,
ValidateFunc: validation.StringInSlice([]string{"CODE", "TOKEN", "SAML2BEARER", "PASSWORD", "CLIENT-CREDENTIALS"}, false),
ValidateFunc: validation.StringInSlice([]string{"CODE", "TOKEN", "SAML2BEARER", "SAML2-BEARER", "PASSWORD", "CLIENT-CREDENTIALS"}, false),
},
"scopes": {
Description: "The scopes requested by this client. Scopes must be set for clients not using the CLIENT-CREDENTIALS grant.",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,13 @@ func readOutboundContactListContact(ctx context.Context, d *schema.ResourceData,
cp = getContactProxy(sdkConfig)
)

contactListId := d.Get("contact_list_id").(string)
contactId := d.Get("contact_id").(string)
contactListId, contactId := splitComplexContact(d.Id())
if contactListId == "" {
contactListId = d.Get("contact_list_id").(string)
}
if contactId == "" {
contactId = d.Get("contact_id").(string)
}

cc := consistency_checker.NewConsistencyCheck(ctx, d, meta, ResourceOutboundContactListContact(), constants.DefaultConsistencyChecks, resourceName)

Expand All @@ -95,6 +100,7 @@ func readOutboundContactListContact(ctx context.Context, d *schema.ResourceData,
}

_ = d.Set("contact_list_id", *contactResponseBody.ContactListId)
_ = d.Set("contact_id", *contactResponseBody.Id)
resourcedata.SetNillableValue(d, "callable", contactResponseBody.Callable)
resourcedata.SetNillableValue(d, "data", contactResponseBody.Data)
resourcedata.SetNillableValueWithSchemaSetWithFunc(d, "phone_number_status", contactResponseBody.PhoneNumberStatus, flattenPhoneNumberStatus)
Expand All @@ -103,7 +109,7 @@ func readOutboundContactListContact(ctx context.Context, d *schema.ResourceData,
return cc.CheckState(d)
})
if retryErr != nil {
return util.BuildAPIDiagnosticError(resourceName, fmt.Sprintf("failed to read contact by ID '%s' from contact list '%s'. Error: %v", d.Id(), contactListId, contactId), resp)
return util.BuildAPIDiagnosticError(resourceName, fmt.Sprintf("failed to read contact by ID '%s' from contact list '%s'. Error: %v", contactId, contactListId, retryErr), resp)
}
log.Printf("Done reading contact '%s' in contact list '%s'", contactId, contactListId)
return nil
Expand All @@ -114,8 +120,13 @@ func updateOutboundContactListContact(ctx context.Context, d *schema.ResourceDat
cp := getContactProxy(sdkConfig)

contactRequestBody := buildDialerContactFromResourceData(d)
contactListId := d.Get("contact_list_id").(string)
contactId := d.Get("contact_id").(string)
contactListId, contactId := splitComplexContact(d.Id())
if contactListId == "" {
contactListId = d.Get("contact_list_id").(string)
}
if contactId == "" {
contactId = d.Get("contact_id").(string)
}

log.Printf("Updating contact '%s' in contact list '%s'", contactId, contactListId)
_, resp, err := cp.updateContact(ctx, contactListId, contactId, contactRequestBody)
Expand All @@ -132,8 +143,13 @@ func deleteOutboundContactListContact(ctx context.Context, d *schema.ResourceDat
sdkConfig := meta.(*provider.ProviderMeta).ClientConfig
cp := getContactProxy(sdkConfig)

contactListId := d.Get("contact_list_id").(string)
contactId := d.Get("contact_id").(string)
contactListId, contactId := splitComplexContact(d.Id())
if contactListId == "" {
contactListId = d.Get("contact_list_id").(string)
}
if contactId == "" {
contactId = d.Get("contact_id").(string)
}

log.Printf("Deleting contact '%s' from contact list '%s'", contactId, contactListId)
resp, err := cp.deleteContact(ctx, contactListId, contactId)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -140,14 +140,14 @@ Only applicable on the creation of a contact, so updating this field will force
Description: `A map of phone number columns to PhoneNumberStatuses, which indicate if the phone number is callable or not.`,
Type: schema.TypeSet,
Optional: true,
ForceNew: true,
Computed: true,
Elem: phoneNumberStatusResource,
},
"contactable_status": {
Description: `A map of media types (Voice, SMS and Email) to ContactableStatus, which indicates if the contact can be contacted using the specified media type.`,
Type: schema.TypeSet,
Optional: true,
ForceNew: true,
Computed: true,
Elem: contactableStatusResource,
},
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,12 @@ func buildWritableContactFromResourceData(d *schema.ResourceData) platformclient
contactRequest.Data = &stringMap
}

contactRequest.PhoneNumberStatus = buildPhoneNumberStatus(d)
contactRequest.ContactableStatus = buildContactableStatus(d)
if phoneNumberStatus := buildPhoneNumberStatus(d); phoneNumberStatus != nil {
contactRequest.PhoneNumberStatus = phoneNumberStatus
}
if contactableStatus := buildContactableStatus(d); contactableStatus != nil {
contactRequest.ContactableStatus = contactableStatus
}
return contactRequest
}

Expand All @@ -47,14 +51,18 @@ func buildDialerContactFromResourceData(d *schema.ResourceData) platformclientv2
stringMap := utillists.ConvertMapStringAnyToMapStringString(dataMap)
contactRequest.Data = &stringMap
}
contactRequest.PhoneNumberStatus = buildPhoneNumberStatus(d)
contactRequest.ContactableStatus = buildContactableStatus(d)
if phoneNumberStatus := buildPhoneNumberStatus(d); phoneNumberStatus != nil {
contactRequest.PhoneNumberStatus = phoneNumberStatus
}
if contactableStatus := buildContactableStatus(d); contactableStatus != nil {
contactRequest.ContactableStatus = contactableStatus
}
return contactRequest
}

func buildContactableStatus(d *schema.ResourceData) *map[string]platformclientv2.Contactablestatus {
contactableStatus, ok := d.Get("contactable_status").(*schema.Set)
if !ok {
if !ok || len(contactableStatus.List()) == 0 {
return nil
}

Expand Down Expand Up @@ -89,7 +97,7 @@ func buildContactableStatus(d *schema.ResourceData) *map[string]platformclientv2

func buildPhoneNumberStatus(d *schema.ResourceData) *map[string]platformclientv2.Phonenumberstatus {
phoneNumberStatus, ok := d.Get("phone_number_status").(*schema.Set)
if !ok {
if !ok || len(phoneNumberStatus.List()) == 0 {
return nil
}

Expand Down Expand Up @@ -192,3 +200,11 @@ func GenerateColumnStatus(column, contactable string) string {
func createComplexContact(contactListId, contactId string) string {
return fmt.Sprintf("%s:%s", contactListId, contactId)
}

func splitComplexContact(complexContact string) (string, string) {
if strings.Contains(complexContact, ":") {
split := strings.SplitN(complexContact, ":", 2)
return split[0], split[1]
}
return "", complexContact
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,13 @@ package responsemanagement_response

import (
"fmt"
"github.com/google/uuid"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
"terraform-provider-genesyscloud/genesyscloud/provider"
respmanagementLibrary "terraform-provider-genesyscloud/genesyscloud/responsemanagement_library"
"terraform-provider-genesyscloud/genesyscloud/util"
"testing"

"github.com/google/uuid"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
)

func TestAccDataSourceResponsemanagementResponse(t *testing.T) {
Expand Down Expand Up @@ -43,6 +44,7 @@ func TestAccDataSourceResponsemanagementResponse(t *testing.T) {
generateTextsBlock(
textsContent,
textsContentTypes[0],
util.NullValue,
),
) + generateResponsemanagementResponseDataSource(
responseData,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
package responsemanagement_response

import (
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
"terraform-provider-genesyscloud/genesyscloud/provider"
resourceExporter "terraform-provider-genesyscloud/genesyscloud/resource_exporter"
registrar "terraform-provider-genesyscloud/genesyscloud/resource_register"

"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
)

/*
Expand Down Expand Up @@ -39,6 +40,12 @@ var (
Type: schema.TypeString,
ValidateFunc: validation.StringInSlice([]string{`text/plain`, `text/html`}, false),
},
`type`: {
Description: `Response text type.`,
Optional: true,
Type: schema.TypeString,
ValidateFunc: validation.StringInSlice([]string{`body`, `subject`}, false),
},
},
}

Expand Down
Loading

0 comments on commit 51bf7f5

Please sign in to comment.