Skip to content

Polish README FAQ copy for leaf-node auth callout behavior#28

Merged
aricart merged 2 commits into
mainfrom
copilot/fix-review-comments
Jun 2, 2026
Merged

Polish README FAQ copy for leaf-node auth callout behavior#28
aricart merged 2 commits into
mainfrom
copilot/fix-review-comments

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 2, 2026

This updates the README FAQ added in PR #27 to address all review-thread feedback: typo fixes, clearer grammar, consistent terminology, and improved readability in the workaround guidance. Scope is limited to the commented section only.

  • Problem statement alignment

    • Applied all requested edits from the linked review thread to the new FAQ content in README.md.
    • Kept changes strictly to wording/formatting issues called out in review.
  • FAQ copy and terminology cleanup

    • Standardized terminology to “leaf node”.
    • Corrected wording and typos (principle, Note that, if, Connecting, and optionally rename).
    • Rephrased awkward sentences for clearer user-facing guidance without changing technical intent.
  • Workaround list readability

    • Added punctuation/flow improvements (including by:) and consistent bullet style.
    • Kept both documented workaround options intact while making each item easier to scan.
This block cannot be disabled, but it can be circumvented by:
- Connecting the auth callout accounts on the leaves to a dedicated non-auth-callout-enabled account on the hub.
- Using import/export on the leaf node to export (and optionally rename) `$SYS.REQ.USER.AUTH` before routing to the hub.

Copilot AI changed the title [WIP] Fix code based on review comments Polish README FAQ copy for leaf-node auth callout behavior Jun 2, 2026
Copilot AI requested a review from aricart June 2, 2026 15:30
@aricart aricart marked this pull request as ready for review June 2, 2026 15:32
@aricart aricart merged commit aab6344 into main Jun 2, 2026
2 checks passed
@aricart aricart deleted the copilot/fix-review-comments branch June 2, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants