-
Notifications
You must be signed in to change notification settings - Fork 634
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Excel.cs #14066
base: master
Are you sure you want to change the base?
Update Excel.cs #14066
Conversation
Related to DynamoDS/DynamoWishlist#200
Fix build
Changed to draft PR temporarily while we look into some CICD issues. |
@denizmaral this is an opensource project. For the benefit of everyone it would be extremely useful if you write a description and purpose for this PR if nothing else. |
Please see the Feature request DynamoDS/DynamoWishlist#200 |
hi @aparajit-pratap Do you know the latest state on this PR? Is this something we could ship now? |
I think the changes in this PR need to be moved from DSOffice/Excel.cs to DSOfficeUtilities/Excel.cs |
I will work with the team and try to get this shipped in Global launch this year |
Please Note:
DynamoRevit
repo will need to be cherry-picked into all the DynamoRevit Release branches that Dynamo supports. Contributors will be responsible for cherry-picking their reviewed commits to the other branches after aLGTM
label is added to the PR.Purpose
Related to DynamoDS/DynamoWishlist#200
Declarations
Check these if you believe they are true
*.resx
filesRelease Notes
(FILL ME IN) Brief description of the fix / enhancement. Mandatory section
Reviewers
(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)
(FILL ME IN, optional) Any additional notes to reviewers or testers.
FYIs
(FILL ME IN, Optional) Names of anyone else you wish to be notified of