Skip to content

Allow to optionally delete data from managed folders and/or job and scenario logs …#188

Merged
pbailly merged 8 commits into
release/10.0from
feature/dss100-sc-71959-delete-job-logs-when-deleting-project
Dec 3, 2021
Merged

Allow to optionally delete data from managed folders and/or job and scenario logs …#188
pbailly merged 8 commits into
release/10.0from
feature/dss100-sc-71959-delete-job-logs-when-deleting-project

Conversation

@vojtech2

@vojtech2 vojtech2 commented Nov 1, 2021

Copy link
Copy Markdown
Contributor

…when deleting a project

sc-71959

Also see: dataiku/dip#13848

@shortcut-integration

Copy link
Copy Markdown

This pull request has been linked to Shortcut Story #71959: Ability to delete job logs when deleting project.

@vojtech2 vojtech2 requested a review from pbailly November 3, 2021 16:57

@pbailly pbailly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but don't merge before 10.0.0 release.

Comment thread dataikuapi/dss/project.py Outdated
Co-authored-by: Pierre Bailly-Ferry <pbailly@users.noreply.github.com>
@vojtech2 vojtech2 changed the base branch from master to release/10.0 November 16, 2021 11:23
@vojtech2 vojtech2 requested a review from cstenac November 16, 2021 11:35
@vojtech2 vojtech2 added this to the 10.0.3 milestone Nov 22, 2021
@vojtech2 vojtech2 changed the title Allow to optionally delete data from managed folders and/or job logs … Allow to optionally delete data from managed folders and/or job and scenario logs … Nov 23, 2021
Comment thread dataikuapi/dss/project.py

@pbailly pbailly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pbailly pbailly merged commit 7f461cf into release/10.0 Dec 3, 2021
@vojtech2 vojtech2 deleted the feature/dss100-sc-71959-delete-job-logs-when-deleting-project branch December 9, 2021 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants