-
Notifications
You must be signed in to change notification settings - Fork 153
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
Start normalizing props for DUI3 (CNX-9169) #3385
Conversation
DUI3-DX/Connectors/Rhino/Speckle.Connectors.Rhino7/DependencyInjection/RhinoPlugin.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Connectors/Rhino/Speckle.Connectors.Rhino7/Plugin/Speckle.Connectors.Rhino7Command.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Connectors/Rhino/Speckle.Connectors.Rhino7/Plugin/Speckle.Connectors.Rhino7Plugin.cs
Outdated
Show resolved
Hide resolved
...-DX/Converters/ArcGIS/Speckle.Converters.ArcGIS3/Layers/PointcloudLayerToSpeckleConverter.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Converters/ArcGIS/Speckle.Converters.ArcGIS3/Layers/VectorLayerToHostConverter.cs
Show resolved
Hide resolved
DUI3-DX/DUI3/Speckle.Connectors.DUI/Models/Card/SendFilter/DirectSelectionSendFilter.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Objects/Speckle.Objects.Common/Speckle.Objects.Common.csproj
Outdated
Show resolved
Hide resolved
DUI3-DX/Sdk/Speckle.Connectors.Utils/Speckle.Connectors.Utils.csproj
Outdated
Show resolved
Hide resolved
DUI3-DX/Sdk/Speckle.Converters.Common.DependencyInjection/ToHost/ToHostConverterWithFallback.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Connectors/Rhino/Speckle.Connectors.Rhino7/Plugin/Speckle.Connectors.Rhino7Plugin.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Connectors/Rhino/Speckle.Connectors.Rhino7/Bindings/RhinoBasicConnectorBinding.cs
Outdated
Show resolved
Hide resolved
...-DX/Converters/ArcGIS/Speckle.Converters.ArcGIS3/Layers/PointcloudLayerToSpeckleConverter.cs
Outdated
Show resolved
Hide resolved
updated again @JR-Morgan 😬 |
...Connectors/Autocad/Speckle.Connectors.AutocadShared/Bindings/AutocadBasicConnectorBinding.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Connectors/Revit/Speckle.Connectors.RevitShared/Bindings/SelectionBinding.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Connectors/Revit/Speckle.Connectors.RevitShared/Bindings/SendBinding.cs
Outdated
Show resolved
Hide resolved
DUI3-DX/Connectors/Rhino/Speckle.Connectors.Rhino7/Bindings/RhinoBasicConnectorBinding.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it dup with Speckle.Connectors.Utils
? seems to me general enough to have it on Core
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is off the alpha branch. I didn't make a new project?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This class is duped between the two projects.
After discussing this a little with Adam, we're thinking this duplication will be resolved when we look at changing our common/sdk projects.
I don't think we should pollute Core
with opinionated, non essential, non-API related features
Moving stuff from csprojs to props file for better management