Skip to content

AI4S Docs Update#806

Merged
ashishguptaiitb merged 2 commits into
mainfrom
ai-for-service-dev
Jul 16, 2026
Merged

AI4S Docs Update#806
ashishguptaiitb merged 2 commits into
mainfrom
ai-for-service-dev

Conversation

@AbhinabDu-Kore

Copy link
Copy Markdown
Collaborator

No description provided.

@AbhinabDu-Kore
AbhinabDu-Kore marked this pull request as ready for review July 16, 2026 15:44
@ashishguptaiitb
ashishguptaiitb merged commit 1e33f99 into main Jul 16, 2026
3 checks passed

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The heading level of some headings at the end of the articles seem to be mistakenly increased. In the on-page TOC, these headings are now H2s and at par with "Connecting your network...". In fact, this heading is empty now.

Image

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

List of ports is missing from SG section. Check the engineering inputs for it. It not provided previously, then ask for it now and update it please.
https://docs.kore.ai/ai-for-service/channels/voice-gateway/deployment-and-operations#singapore-sg-region

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Format of documenting these details is different for SG region - tabulated and condense. For other regions, the list is flat section with short tables. See https://docs.kore.ai/ai-for-service/channels/voice-gateway/deployment-and-operations#singapore-sg-region
The former approach is better to shorten the article. Also, SG region's table can be a bit more optimized. It seems AI-generated but a better prompt can make it more condensed (less row).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

https://docs.kore.ai/ai-for-service/channels/voice-gateway/deployment-and-operations#infrastructure-setup-guide <- the section can be remove and its content can be merged in the beginning, after shortening it.

@ashishguptaiitb ashishguptaiitb Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

https://docs.kore.ai/ai-for-service/channels/voice-gateway/minimum-system-requirements-and-supported-browsers is a copy-paste without edits! Needs to be completely re-formatted for 1. better style and 2. shorter length. Both articles contain "technical requirements", then why keep them separate?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Inline with our new writing style - short and crisp sentences, consider using AI tool to shorten the content of verbose sections such as this one,
Image

@ashishguptaiitb ashishguptaiitb Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Firewall requirements listed in the other article here does not contain SG region. Ask the team for all regions + merge that info here. Scattered requirements across 2 URLs is not a good user experience.

Also, even if it is separate, the sequence of the regions must be the same to make it easy to scan.

@ashishguptaiitb ashishguptaiitb added the PendingFixes Some PR review comments are yet to be resolved. label Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PendingFixes Some PR review comments are yet to be resolved.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants