Skip to content
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

[DAR-1657][External] Introduce by-passable warning when an import will overwrite annotations #835

Merged
merged 6 commits into from
May 7, 2024

Conversation

JBWilkie
Copy link
Collaborator

@JBWilkie JBWilkie commented May 6, 2024

Problem

When importing annotations to a dataset item with existing annotations, unless append is set to True, the current annotations will be overwritten. Several users have been inconvenienced by the lack of warning when doing this

Solution

Introduce a by-passable warning that presents users with the potential impact of their import if any item targeted in the import has annotations. The warning is not displayed if:

  • No item targeted by the import has annotations, or:
  • The --overwrite option is set to True when executing the import

Changelog

Introduced a by-passable warning when performing an annotation import that would overwrite existing annotations

Copy link

linear bot commented May 6, 2024

@JBWilkie JBWilkie merged commit 112a857 into master May 7, 2024
16 checks passed
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