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

Allow creating a saved annotations with the same title #4884

Merged
merged 7 commits into from
Sep 13, 2023

Conversation

jorg-vr
Copy link
Contributor

@jorg-vr jorg-vr commented Aug 10, 2023

This pull request makes it possible to create two saved annotations with the same name.
Instead of crashing on duplicate titles, a warning is now shown.

When saving a new annotation: (The field has a message as title text)
image
With confirm before save:
image

When saving an existing annotation:
image
With confirm before save:
image

When editing an existing saved annotation:
image
I have not done a confirm before save on this page. (I can do that but that would probably involve reconverting the page to a webcomponent and reusing the form component to do it properly)

Part of #4564

@jorg-vr jorg-vr added the enhancement A change that isn't substantial enough to be called a feature label Aug 10, 2023
@jorg-vr jorg-vr requested a review from a team as a code owner August 10, 2023 11:06
@jorg-vr jorg-vr self-assigned this Aug 10, 2023
@jorg-vr jorg-vr requested review from bmesuere and chvp and removed request for a team August 10, 2023 11:06
@jorg-vr jorg-vr marked this pull request as draft August 10, 2023 11:07
@jorg-vr jorg-vr marked this pull request as ready for review August 17, 2023 13:57
@jorg-vr jorg-vr merged commit 5fbbf71 into main Sep 13, 2023
12 of 14 checks passed
@jorg-vr jorg-vr deleted the enhance/allow-duplicate-names branch September 13, 2023 07:27
@jorg-vr jorg-vr temporarily deployed to naos September 13, 2023 07:27 — with GitHub Actions Inactive
@jorg-vr jorg-vr temporarily deployed to production September 13, 2023 07:32 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A change that isn't substantial enough to be called a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants