Purpose

Following up on a https://jira.autodesk.com/browse/DYN-5709, this PR addresses a bigger concern. The Publish Version experience has been broken for a while now. After a lengthy research and trying several options, the proposed solution is very simple.

Context - the retain folder structure option was introduced as an alternative way to publish a package. It was modeled after the existing and only way, which lead Dynamo predefine a set of 4 folders (bin, doc, dyf, extra), and allocate packages accordingly. As one of the steps, this flow would check and delete pre-existing files in the above locations before copying the files to a finalize package structure. This made sense, as the files would come from a separate place to be copied in this new package folder structure.

In contrast, the retain folder structure takes files and folders exactly as they are, and copies them to the default package folder, so the final package is loaded in Dynamo and ready to use. However, in case there is a folder already installed locally, and the user needs to submit it for the first time online, or submit it as a new version, the source files will be the same as the target files. In this case, it makes no sense to try to override the file with the same file, so we simply skip it.

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

• fixing the publish version workflow by removing the deletion of existing files from the 'retain folder structure' route
• a few minor issues found, fix

Reviewers

@aparajit-pratap
@reddyashish
@QilongTang
@zeusongit

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of