Conversation
Solutions/Visa Threat Intelligence (VTI)/Package/mainTemplate.json
Outdated
Show resolved
Hide resolved
| "_dataConnectorContentIdConnections": "VisaThreatIntelligenceConnectorTemplateConnections", | ||
| "dataConnectorTemplateNameConnections": "[concat(parameters('workspace'),'-dc-',uniquestring(variables('_dataConnectorContentIdConnections')))]", | ||
| "_logAnalyticsTableId1": "VisaThreatIntelligenceIOC_CL", | ||
| "_solutionAuthor": "Visa Inc.", |
There was a problem hiding this comment.
update values
"name": "[variables('_solutionAuthor')]",
"tier": "[variables('_solutionTier')]",
There was a problem hiding this comment.
These are the places where the variables are set. I don't see any other places where we define these variables
There was a problem hiding this comment.
Pull request overview
Regenerates the Visa Threat Intelligence (VTI) solution package artifacts to address a template validation failure and bumps solution version metadata.
Changes:
- Added regenerated ARM package templates and UI definition under
Solutions/Visa Threat Intelligence (VTI)/Package/. - Added/updated solution definition file
Solutions/Visa Threat Intelligence (VTI)/Data/Solution_VTI.jsonwith solution metadata, version, and content lists.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| Solutions/Visa Threat Intelligence (VTI)/Package/testParameters.json | Package artifact (ignored for review per repo guidelines). |
| Solutions/Visa Threat Intelligence (VTI)/Package/mainTemplate.json | Package artifact (ignored for review per repo guidelines). |
| Solutions/Visa Threat Intelligence (VTI)/Package/createUiDefinition.json | Package artifact (ignored for review per repo guidelines). |
| Solutions/Visa Threat Intelligence (VTI)/Data/Solution_VTI.json | Defines solution metadata and component inventory; needs adjustments to meet solution-data validation rules. |
| @@ -0,0 +1,29 @@ | |||
| { | |||
| "Name": "Visa Threat Intelligence (VTI)", | |||
There was a problem hiding this comment.
The solution Name contains special characters ( and ) which violates the solution-data naming requirement (only alphanumeric characters and spaces). Update the display name to remove special characters (for example, “Visa Threat Intelligence VTI”) while keeping it aligned with the solution folder naming expectations.
Solutions/Visa Threat Intelligence (VTI)/Data/Solution_VTI.json
Outdated
Show resolved
Hide resolved
Solutions/Visa Threat Intelligence (VTI)/Data/Solution_VTI.json
Outdated
Show resolved
Hide resolved
Solutions/Visa Threat Intelligence (VTI)/Data/Solution_VTI.json
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@sreedharande @v-shukore @v-maheshbh The PR check is stuck from yesterday |

Required items, please complete
Change(s):
Reason for Change(s):
Version Updated:
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present: