Skip to content

Rejection inbound samples#35

Open
nnpvaan wants to merge 4 commits intosenaite:2.xfrom
nnpvaan:reject-inbound-samples
Open

Rejection inbound samples#35
nnpvaan wants to merge 4 commits intosenaite:2.xfrom
nnpvaan:reject-inbound-samples

Conversation

@nnpvaan
Copy link
Contributor

@nnpvaan nnpvaan commented May 9, 2025

Description of the issue/feature this PR addresses

Implements support for rejecting InboundSample objects with assigned rejection reasons. This enhancement ensures users can provide reasons for rejecting inbound samples and allows batch rejection through a dedicated view

Current behavior before PR

  • InboundSample objects could be rejected without specifying rejection reasons
  • No dedicated view to enter rejection reasons for multiple samples
  • Do not show viewlet for rejected samples

Desired behavior after PR is merged

  • InboundSample type includes a new field selected_rejection_reasons and other_rejection_reasons
  • Clicking "Reject" in the Inbound Sample listing redirects users to a view where they can enter rejection reasons for one or more samples
image
  • Show reasons in viewlet for rejected inbound samples
image

--
I confirm I have tested this PR thoroughly and coded it according to PEP8
and Plone's Python styleguide standards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant