Skip to content

Added a confirmation dialog for deleting/cleaning a model. - #525

Merged
chenejac merged 1 commit into
vivo-project:mainfrom
litvinovg:VIVO-1045
Jun 20, 2026
Merged

Added a confirmation dialog for deleting/cleaning a model.#525
chenejac merged 1 commit into
vivo-project:mainfrom
litvinovg:VIVO-1045

Conversation

@litvinovg

Copy link
Copy Markdown
Member

VIVO GitHub issue

What does this pull request do?

Added a confirmation dialog for deleting/cleaning a model.

How should this be tested?

Interested parties

@VIVO-project/vivo-committers

Reviewers' expertise

Candidates for reviewing this PR should have some of the following expertises:

  1. Java
  2. HTML
  3. JavaScript
  4. JSP

Reviewers' report template

General comment

A reviewer should provide here comments and suggestions for requested changes if any.

Testing

A reviewer should briefly describe here how it was tested

Code reviewing

A reviewer should briefly describe here which part was code reviewed

@brianjlowe brianjlowe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good; long-needed improvement.

@chenejac chenejac 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.

@litvinovg works as described. The code is clean. Thanks

@chenejac
chenejac merged commit 8821a50 into vivo-project:main Jun 20, 2026
1 check 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.

VIVO-1045: "Manage Jena Models" page should have safeguards

3 participants