Conversation
There was a problem hiding this comment.
PR Summary:
This PR adds compatibility for the Autodesk Desktop Connector API with different .NET versions. It addresses an issue with the keynote manager and ADC module by selecting the appropriate API path based on Revit version - using the Civil3D subfolder for Revit 2025+ (.NET 8 Core) and the standard path for older versions (.NET Framework 4).
Review Summary:
The implementation effectively addresses the compatibility issue with a clean, targeted approach. I've identified a couple of areas for improvement related to error handling and managing installation path assumptions. These changes would make the code more robust across different environments and installation configurations.
Feel free to provide feedback on this review for future improvements.
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25101+0700-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25101+1501-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+0813-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1015-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1041-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25104+1127-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25105+0957-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+0734-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1252-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1322-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25106+1408-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25107+0621-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25116+1412-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25118+1113-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25119+1928-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25133+1246-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0631-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25136+0639-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25140+1615-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25141+1904-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25143+0904-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25146+0559-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1012-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25149+1943-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25153+1459-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0858-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25154+0920-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25155+0904-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1125-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25162+1306-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25162+2030-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+0700-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1347-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1420-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25164+1830-wip |
|
📦 New work-in-progress (wip) builds are available for 5.1.0.25171+0757-wip |
|
📦 New work-in-progress (wip) builds are available for 5.2.0.25181+1313-wip |
|
📦 New public release are available for 5.2.0.25181+1425 |
Switch for .net core Desktop connector API
Description
adc module and keynote manager use the desktop connector API, .net framework 4) works below 2024
Revit 2025-> .net 8 core -> using API from one of the desktop connector subfolder for Civil3D
Checklist
Before submitting your pull request, ensure the following requirements are met:
pipenv run black {source_file_or_directory}Related Issues