Skip to content

Cherry-Pick-Html localization to master (4 commits)#15978

Merged
QilongTang merged 13 commits intoDynamoDS:masterfrom
RobertGlobant20:html_localization_to_master
Mar 25, 2025
Merged

Cherry-Pick-Html localization to master (4 commits)#15978
QilongTang merged 13 commits intoDynamoDS:masterfrom
RobertGlobant20:html_localization_to_master

Conversation

@RobertGlobant20
Copy link
Copy Markdown
Contributor

Purpose

Cherry-Pick-Html localization to master (4 commits)
In this PR there is one xml that doesn't exist yet DSCoreNodes.xml, the file will exist when the next PR is merged to master #15977

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • 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
  • All tests pass using the self-service CI.
  • 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

Release Notes

Cherry-Pick-Html localization to master (4 commits)

Reviewers

@QilongTang

FYIs

@ZbynekHanak-RWS

@QilongTang
Copy link
Copy Markdown
Contributor

@RobertGlobant20 I made a try to fix the build, let me know if it's the right fix or not

QilongTang and others added 5 commits March 25, 2025 09:23
I'm removing the localized resx files as embedded resources from DynamoCoreWpf (due that are missing) and also after the merge there was an extra  </Target> which was removed.
re-triggering the build/jobs
@QilongTang
Copy link
Copy Markdown
Contributor

@RobertGlobant20 Curious, still in the process of making more changes?

@RobertGlobant20
Copy link
Copy Markdown
Contributor Author

@RobertGlobant20 Curious, still in the process of making more changes?

No, I'm done with the changes, now all jobs/checks passed.

Copy link
Copy Markdown
Contributor

@QilongTang QilongTang left a comment

Choose a reason for hiding this comment

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

LGTM, let's give it a try

@QilongTang QilongTang merged commit 1843deb into DynamoDS:master Mar 25, 2025
16 of 17 checks passed
RobertGlobant20 added a commit to RobertGlobant20/Dynamo that referenced this pull request Mar 26, 2025
Co-authored-by: Aaron (Qilong) <aaron.tang@autodesk.com>
Co-authored-by: Aaron (Qilong) <173288704@qq.com>
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