-
Notifications
You must be signed in to change notification settings - Fork 1
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
Neat 206 delete dms data model step is missing in v2 rules #421
Neat 206 delete dms data model step is missing in v2 rules #421
Conversation
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
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 good. As we discussed yesterday, the support for item and not just id should be in the base class of the loader, but that can be later.
If you have time, an integration test would be great for this.
@@ -87,8 +87,10 @@ def retrieve(self, ids: SequenceNotStr[str]) -> SpaceList: | |||
def update(self, items: Sequence[SpaceApply]) -> SpaceList: | |||
return self.create(items) | |||
|
|||
def delete(self, ids: SequenceNotStr[str]) -> list[str]: | |||
return self.client.data_modeling.spaces.delete(ids) | |||
def delete(self, ids: SequenceNotStr[str] | Sequence[Space | SpaceApply]) -> list[str]: |
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.
Suggest adding a technical debt task (unless you want to tackle it immediately) on moving this to the base class.
[0.75.7] - 29-05-24
Added
DMSExporter
now supports deletion of data model and data model componentsDeleteDataModelFromCDF
added to the step library