Skip to content

[docs] Clarify connection property names and alias fallback examples - #1785

Open
Sébastien Ros (sebastienros) wants to merge 4 commits into
release/13.6from
sebros/connection-string-alias-docs
Open

Sébastien Ros (sebastienros) wants to merge 4 commits into
release/13.6from
sebros/connection-string-alias-docs

Conversation

@sebastienros

Copy link
Copy Markdown
Contributor

Summary

Follow up on the review feedback in #1778, which merged before these local fixes were committed.

  • Move the Radius naming explanation into Resource properties and use a table to distinguish password property names from complete connection-string aliases.
  • Make the Python and JavaScript examples try the original name before the portable alias, matching the C# example and preserving explicitly empty values.

Only src/frontend/src/content/docs/fundamentals/environment-variables.mdx changes relative to release/13.6 (+16/-4). Preserves the wording changes made when #1778 merged. Existing review threads are left unresolved.

Validation

  • Executed the Python and JavaScript snippets with both aliases, portable-only, an empty original value, and neither alias.
  • MDX compilation and git diff --check passed.

No product code, configuration, or dependency changes.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Follow up on review feedback from #1778.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

# Conflicts:
#	src/frontend/src/content/docs/fundamentals/environment-variables.mdx
@sebastienros

Copy link
Copy Markdown
Contributor Author

A previous PR got merged before I could react to comments

@aspire-repo-bot

Copy link
Copy Markdown
Contributor

Frontend HTML artifact ready

The latest frontend build uploaded the frontend-dist artifact for PR #1785. Use the VS Code button below to open this PR with GitHub Artifacts Explorer and browse the built HTML locally.

VS Code: Open PR #1785 artifacts

This comment updates automatically when a new frontend build artifact is uploaded.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

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