Skip to content

Add reply-back action to org application info requests#19594

Open
Ayudh-M wants to merge 6 commits intopypi:mainfrom
Ayudh-M:fix-18040-reply-back-button
Open

Add reply-back action to org application info requests#19594
Ayudh-M wants to merge 6 commits intopypi:mainfrom
Ayudh-M:fix-18040-reply-back-button

Conversation

@Ayudh-M
Copy link

@Ayudh-M Ayudh-M commented Mar 2, 2026

Pull Request Checklist

  • I’ve read the guidelines for contributing.
  • I created a dedicated branch fix-18040-reply-back-button.
  • I ran the relevant unit test(s) and got no failures.

Description — fix #18040

Added a Reply Back button near the bottom of the Information Requests conversation on the admin organization application detail page. The button reuses the existing Request More Info modal (#requestMoreInfoModal) and its existing POST handler.

This patch:

  • adds a bottom-of-thread Reply Back button that opens the existing #requestMoreInfoModal
  • mirrors existing disable logic (permission + status checks) used by the top Request Info action
  • adds a small regression unit test to ensure the modal trigger exists twice (top + bottom) and the new label is present

No functional change beyond exposing the existing “request more info” action at the bottom of the conversation.

Fixes #18040

Testing

command expected / observed
python -m pytest tests/unit/admin/test_organization_application_detail_template.py passes
manual UI: admin org application detail → scroll to bottom of “Information Requests” Reply Back present and opens Request More Info modal

@Ayudh-M Ayudh-M requested a review from a team as a code owner March 2, 2026 05:18
@miketheman
Copy link
Member

@Ayudh-M You might want to run make reformat , make lint, make translations locally to prevent the back-and-forth with CI.
https://warehouse.pypa.io/development/submitting-patches/#code

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.

Add a "Reply Back" button for Org Applications

2 participants