Skip to content

Dyn-9003 - Update AutocompleteLowConfidenceMessage#16298

Merged
johnpierson merged 1 commit into
DynamoDS:masterfrom
johnpierson:dyn9003_reworkLowConfidenceLanguage
Jun 20, 2025
Merged

Dyn-9003 - Update AutocompleteLowConfidenceMessage#16298
johnpierson merged 1 commit into
DynamoDS:masterfrom
johnpierson:dyn9003_reworkLowConfidenceLanguage

Conversation

@johnpierson

@johnpierson johnpierson commented Jun 13, 2025

Copy link
Copy Markdown
Member

Purpose

Updates the user-facing message for low-confidence autocomplete suggestions to be more user-friendly and encouraging.

Refines the text of AutocompleteLowConfidenceMessage in both default and en-US resource files.
Streamlines messaging to remove references to preference adjustments.

image

Declarations

Check these if you believe they are true

  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated
  • This PR contains no files larger than 50 MB
  • This PR introduces new feature code involve network connecting and is tested with no-network mode.

Release Notes

N/A

Reviewers

@DynamoDS/synapse

FYIs

@achintyabhat @emru-y

- revise AutocompleteLowConfidenceMessage to "No confident suggestions available. The model is continuously improving - check back soon."
Copilot AI review requested due to automatic review settings June 13, 2025 14:54

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-9003

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

Updates the user-facing message for low-confidence autocomplete suggestions to be more user-friendly and encouraging.

  • Refines the text of AutocompleteLowConfidenceMessage in both default and en-US resource files.
  • Streamlines messaging to remove references to preference adjustments.

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/DynamoCore/Properties/Resources.resx Updated AutocompleteLowConfidenceMessage
src/DynamoCore/Properties/Resources.en-US.resx Updated AutocompleteLowConfidenceMessage
Files not reviewed (1)
  • src/DynamoCore/Properties/Resources.Designer.cs: Language not supported
Comments suppressed due to low confidence (3)

src/DynamoCore/Properties/Resources.resx:891

  • [nitpick] For grammatical consistency, consider adding “are” after “suggestions”: “No confident suggestions are available. The model is continuously improving — check back soon.” Using an em dash also enhances readability.
<value>No confident suggestions available. The model is continuously improving - check back soon.</value>

src/DynamoCore/Properties/Resources.en-US.resx:888

  • [nitpick] Mirror the punctuation change here as well (use em dash) and include “are” for clarity: “No confident suggestions are available. The model is continuously improving — check back soon.”
<value>No confident suggestions available. The model is continuously improving - check back soon.</value>

src/DynamoCore/Properties/Resources.resx:891

  • Ensure that other localized .resx files are updated with the same revised message to keep translations in sync.
<value>No confident suggestions available. The model is continuously improving - check back soon.</value>

@johnpierson

Copy link
Copy Markdown
Member Author

Pretty little PR to just revise the resource, we have another ticket to add this to the new DNA flyout.
https://jira.autodesk.com/browse/DYN-9045

@johnpierson johnpierson merged commit f1b7a16 into DynamoDS:master Jun 20, 2025
34 of 42 checks passed
@johnpierson johnpierson deleted the dyn9003_reworkLowConfidenceLanguage branch June 20, 2025 15:34
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.

3 participants