Add deletion request handling and update related components

[NishantSinghhhhh]

Closes #10033

Feature: Community Deletion Queue

This PR extends the merge queue system to support deletion requests, allowing librarians to request removal of entries that are not books (e.g., calendars, magazines, etc.) in a structured, reviewable way.

Technical

Backend Changes:

• Added DELETION (type 3) to CommunityEditsQueue.TYPE in openlibrary/core/edits.py
• Extended MODES dictionary to include deletion_open and deletion_closed modes
• Updated where_clause() method to filter deletion requests by mr_type = 3
• Modified openlibrary/plugins/upstream/edits.py:
  • Added deletion counts in GET() method (deletion_open, deletion_closed)
  • Updated create_url() to generate deletion URLs (/works/{olid}/delete)
  • Updated create_title() to fetch titles for deletion requests
  • Added DELETION type to needs_unique_url() validation

Frontend Changes:

• Updated merge_request_table.html to display "Community Deletion Requests" title and pass is_deletion_mode flag
• Enhanced table_header.html with two new tabs: "Deletion Open" and "Deletion Closed"
• Modified status filter labels to show "Removed"/"Kept" for deletion mode instead of "Merged"/"Declined"
• Updated table_row.html to display deletion-specific UI:
  • "Deletion:" badge on request titles
  • Different tooltips and messages for deletion requests
  • Updated comment placeholders and submission text
• Added DELETION: 3 to JavaScript REQUEST_TYPES
• Created createDeletionRequest(), approveDeletionRequest(), and declineDeletionRequest() helper functions

Key Implementation Notes:

• Deletion requests only require a single OLID (unlike merges which need multiple)
• Deletion requests are filtered separately from merge requests using mr_type field
• Reuses existing approval/decline workflow - no new database schema changes needed
• Maintains backwards compatibility with existing merge request functionality

Testing

Create a Deletion Request:

curl -X POST http://localhost:8080/merges \
  -H "Content-Type: application/json" \
  -H "Cookie: session=YOUR_SESSION" \
  -d '{"rtype":"create-request","action":"create-pending","mr_type":3,"olids":"OL123W","comment":"This is a calendar, not a book"}'

Screenshot

Screencast.from.2025-11-07.23-18-18.webm

Stakeholders

@RayBB @seabelis

[NishantSinghhhhh]

@RayBB , please review this PR, also do tell me if any more testing is required and

I justed wanted to confirm whether the frontedn changes done by me are acceptable or I should be making some changes in it

[RayBB]

@NishantSinghhhhh I'm not looking at the code much yet but first there's an error in the CI that precommit is giving about nested ifs. Please address that (and always check the CI logs on PRs)

Second, please make sure your pull request only has relevant changes. For example, there are some changes to white space at the end of files. That should not be there.

Finally, to make this useful and easy to test please add a UI for requesting deletes to the librarian toolbar. Here's a similar PR for that #7648

Overall, on first glance this looks like it is the right direction so keep it up!

[NishantSinghhhhh]

@RayBB , I have added the changes, and precommit errors have been removed, I have removed any irrelevant changes, Now sicne last 2 days I am trying to find the librarian toolbar, and I have failed, so can you please tell me at which route or which place I can find the librarian toolbar ??

[RayBB]

@NishantSinghhhhh You can find some of the librarian toolbar here #7502

[NishantSinghhhhh]

@NishantSinghhhhh You can find some of the librarian toolbar here #7502

Thank you, I will add the changes now

[accesslint]

There are accessibility issues in these changes.

[accesslint]

There are accessibility issues in these changes.

[accesslint]

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

[accesslint]

There are accessibility issues in these changes.

[accesslint]

Looks like this element is missing an accessible name or label. That makes it hard for people using screen readers or voice control to use the control.

[accesslint]

There are accessibility issues in these changes.

[accesslint]

There are accessibility issues in these changes.

[NishantSinghhhhh]

Screencast.from.2025-12-30.17-09-52.webm

@jimchamp the Deletion request flow and Request review flow for book is complete .

[NishantSinghhhhh]

Screencast.from.2025-12-30.19-12-08.webm

@jimchamp

While I am working on the Deletion request flow and Request review for author, I am encountering few errros, can you please help me with this

1- I am able to delete authours having 0 books assigned to them, and the Merge Requests also gets closed

2- when I try to create a deletion request for authors having more than 0 books, it shows the error as [bad request ] and "You have a ValidationException. This Author page cannot be deleted as 1 record(s) still reference this id. Please remove or reassign before trying again.", How should I move forward with this.