Skip to content

DYN-9976: Fix typo in Sphere.ByCenterPointRadius and TSplineSurface.ByCenterPointRadius node descriptions#17073

Closed
zeusongit with Copilot wants to merge 1 commit into
masterfrom
copilot/fix-sphere-node-typo
Closed

DYN-9976: Fix typo in Sphere.ByCenterPointRadius and TSplineSurface.ByCenterPointRadius node descriptions#17073
zeusongit with Copilot wants to merge 1 commit into
masterfrom
copilot/fix-sphere-node-typo

Conversation

Copilot AI commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

"cetered" → "centered" in the <summary> doc for two nodes in doc/distrib/xml/en-GB/ProtoGeometry.xml. Typo was visible in both the node tooltip and the Documentation Browser.

  • doc/distrib/xml/en-GB/ProtoGeometry.xml
    • Sphere.ByCenterPointRadius: "cetered""centered"
    • TSplineSurface.ByCenterPointRadius: "cetered""centered"

@zeusongit zeusongit changed the title Fix typo in Sphere.ByCenterPointRadius and TSplineSurface.ByCenterPointRadius node descriptions DYN-9976: Fix typo in Sphere.ByCenterPointRadius and TSplineSurface.ByCenterPointRadius node descriptions Apr 23, 2026

@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-9976

@zeusongit zeusongit marked this pull request as ready for review April 23, 2026 15:47
Copilot AI review requested due to automatic review settings April 23, 2026 15:47

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

Fixes a user-visible typo in the English (en-GB) XML documentation for two ProtoGeometry nodes, improving tooltips and the Documentation Browser text.

Changes:

  • Corrects “cetered” → “centered” in the <summary> for Sphere.ByCenterPointRadius.
  • Corrects “cetered” → “centered” in the <summary> for TSplineSurface.BySphereCenterPointRadius.

@sonarqubecloud

Copy link
Copy Markdown

@aparajit-pratap aparajit-pratap 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.

This file should not be manually edited in Dynamo. It is created while building protogeometry.dll in the libg repo and is sourced from the libg nuget package, then manually checked in again here. This is sort of a roundabout process and should ideally be rectified. The reason this xml file is being submitted here is because the localization team picks up the file from the Dynamo repo for translation. There is a slack conversation about looking at improving this process and moving the xml localization to the source repo, LibG, in this case.

@zeusongit zeusongit closed this Apr 27, 2026
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.

4 participants