Skip to content

fix(help): Display default/configured values when null as well, not just when empty#216

Merged
DecSmith42 merged 2 commits into
mainfrom
fix/help-default-configured-values
Sep 3, 2025
Merged

fix(help): Display default/configured values when null as well, not just when empty#216
DecSmith42 merged 2 commits into
mainfrom
fix/help-default-configured-values

Conversation

@DecSmith42

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings September 3, 2025 05:19
@DecSmith42 DecSmith42 enabled auto-merge September 3, 2025 05:19

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.

Pull Request Overview

This PR fixes a bug in the help service where null values for default and configured parameters were not being handled properly when displaying help information.

  • Converts null values to empty strings for both default and configured parameter values
  • Ensures consistent handling of null values in the help display logic

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread DecSm.Atom/Help/HelpService.cs
Comment thread DecSm.Atom/Help/HelpService.cs
@DecSmith42 DecSmith42 merged commit 9f3feea into main Sep 3, 2025
38 of 39 checks passed
@DecSmith42 DecSmith42 deleted the fix/help-default-configured-values branch September 3, 2025 05:31
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