Skip to content

Conversation

@shalev007
Copy link
Contributor

@shalev007 shalev007 commented Oct 21, 2025

User description

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.

Added docs pages

Please also include the path for the added docs

  • Quickstart (/)
  • Blueprint (/platform-overview/port-components/blueprint)
  • ...

Updated docs pages

Please also include the path for the updated docs

  • Quickstart (/)
  • Blueprint (/platform-overview/port-components/blueprint)
  • ...

PR Type

Documentation


Description

  • Rename AWS integration references to "AWS On-Premise" throughout documentation

  • Update category label and page titles for consistency

  • Standardize terminology across installation and multi-account guides


Diagram Walkthrough

flowchart LR
  A["AWS integration<br/>references"] -- "rename to" --> B["AWS On-Premise<br/>integration"]
  B -- "updates" --> C["Category labels"]
  B -- "updates" --> D["Page titles"]
  B -- "updates" --> E["Installation guides"]
Loading

File Walkthrough

Relevant files
Documentation
Overview.md
Update AWS integration reference terminology                         

docs/build-your-software-catalog/sync-data-to-catalog/cloud-providers/aws-v3/Overview.md

  • Update reference from "self-hosted AWS integration" to "AWS On-Premise
    integration"
+1/-1     
_category_.json
Update category label for consistency                                       

docs/build-your-software-catalog/sync-data-to-catalog/cloud-providers/aws/category.json

  • Change category label from "AWS" to "AWS On-Premise"
+1/-1     
aws.md
Rename AWS to AWS On-Premise throughout                                   

docs/build-your-software-catalog/sync-data-to-catalog/cloud-providers/aws/aws.md

  • Update page title from "AWS" to "AWS On-Premise"
  • Update integration description and section headings
  • Standardize terminology in getting started and mapping sections
+5/-5     
installation.md
Update installation guide terminology                                       

docs/build-your-software-catalog/sync-data-to-catalog/cloud-providers/aws/installations/installation.md

  • Update deployment instructions to reference "AWS On-Premise
    integration"
  • Standardize terminology in permissions and prerequisites sections
+2/-2     
multi_account.md
Update multi-account guide terminology                                     

docs/build-your-software-catalog/sync-data-to-catalog/cloud-providers/aws/installations/multi_account.md

  • Update guide title and description to reference "AWS On-Premise
    Integration"
+1/-1     

@notion-workspace
Copy link

Historical scorecards

@qodo-merge-pro
Copy link
Contributor

qodo-merge-pro bot commented Oct 21, 2025

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
🎫 No ticket provided
- [ ] Create ticket/issue <!-- /create_ticket --create_ticket=true -->

</details></td></tr>
Codebase Duplication Compliance
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
No custom compliance provided

Follow the guide to enable custom compliance check.

  • Update
Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@qodo-merge-pro
Copy link
Contributor

qodo-merge-pro bot commented Oct 21, 2025

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
General
Fix a typo in documentation

Correct the typo "you" to "your" in the documentation.

docs/build-your-software-catalog/sync-data-to-catalog/cloud-providers/aws/installations/installation.md [32]

-The AWS On-Premise integration is deployed using Helm on you cluster.
+The AWS On-Premise integration is deployed using Helm on your cluster.
  • Apply / Chat
Suggestion importance[1-10]: 3

__

Why: The suggestion correctly identifies a typo (you instead of your) on a line modified by the PR, improving the documentation's quality and readability.

Low
Fix a grammatical error in documentation

Correct the grammatical error in the phrase "multiple account's data" to "data
from multiple accounts" or "multiple accounts' data".

docs/build-your-software-catalog/sync-data-to-catalog/cloud-providers/aws/installations/multi_account.md [9]

-This guide will provide you with the necessary tools for enabling our Ocean AWS On-Premise Integration to digest multiple account's data.
+This guide will provide you with the necessary tools for enabling our Ocean AWS On-Premise Integration to digest data from multiple accounts.
  • Apply / Chat
Suggestion importance[1-10]: 3

__

Why: The suggestion correctly identifies a grammatical error ("account's" should be plural possessive "accounts'") and provides a clear fix, improving the documentation's correctness.

Low
  • Update

@aws-amplify-eu-west-1
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-2928.d2ngvl90zqbob8.amplifyapp.com

Copy link
Contributor

@sivanel97 sivanel97 left a comment

Choose a reason for hiding this comment

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

LGTM

@sivanel97 sivanel97 merged commit 8cd2662 into main Oct 21, 2025
5 checks passed
@sivanel97 sivanel97 deleted the PORTN-123-rename-old-aws-to-aws-on-prem-2 branch October 21, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants