Skip to content

Commit

Permalink
Merge branch 'feat/integration_support' into PR_main_integration_support
Browse files Browse the repository at this point in the history
  • Loading branch information
ArpitShukla12 committed Nov 7, 2023
2 parents c57b8ad + 4a63c0c commit fed7a6d
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 14 deletions.
14 changes: 8 additions & 6 deletions adapters/templates/github-integration.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,14 @@ Set your repository action secrets [here](https://github.com/${context.payload.r
export function getSetResourceOnAssetComment(tableMd, setResourceFailed) {
return `## 🎊 Congrats on the merge!
This pull request has been added as a resource to the following assets:
Name | Resource set successfully
--- | ---
${tableMd}
${setResourceFailed ? '> Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}
`
This pull request has been added as a resource to the following assets:
${setResourceFailed ? '> ⚠️ Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}
Name | Resource set successfully
--- | ---
${tableMd}
`
}

export function getAssetInfo(ATLAN_INSTANCE_URL, asset, materialisedAsset, environmentName, projectName) {
Expand Down
4 changes: 3 additions & 1 deletion adapters/templates/gitlab-integration.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,12 @@ export function getSetResourceOnAssetComment(tableMd, setResourceFailed) {
return `## 🎊 Congrats on the merge!
This pull request has been added as a resource to the following assets:
${setResourceFailed ? '> ⚠️ Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}
Name | Resource set successfully
--- | ---
${tableMd}
${setResourceFailed ? '> Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}
`
}

Expand Down
18 changes: 11 additions & 7 deletions dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -25437,12 +25437,14 @@ Set your repository action secrets [here](https://github.com/${context.payload.r
function getSetResourceOnAssetComment(tableMd, setResourceFailed) {
return `## 🎊 Congrats on the merge!

This pull request has been added as a resource to the following assets:
Name | Resource set successfully
--- | ---
${tableMd}
${setResourceFailed ? '> Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}
`
This pull request has been added as a resource to the following assets:

${setResourceFailed ? '> ⚠️ Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}

Name | Resource set successfully
--- | ---
${tableMd}
`
}

function getAssetInfo(ATLAN_INSTANCE_URL, asset, materialisedAsset, environmentName, projectName) {
Expand Down Expand Up @@ -33967,10 +33969,12 @@ function gitlab_integration_getSetResourceOnAssetComment(tableMd, setResourceFai
return `## 🎊 Congrats on the merge!

This pull request has been added as a resource to the following assets:

${setResourceFailed ? '> ⚠️ Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}

Name | Resource set successfully
--- | ---
${tableMd}
${setResourceFailed ? '> Seems like we were unable to set the resources for some of the assets due to insufficient permissions. To ensure that the pull request is linked as a resource, you will need to assign the right persona with requisite permissions to the API token.' : ''}
`
}

Expand Down

0 comments on commit fed7a6d

Please sign in to comment.