Skip to content

Add Release action to Deposits processing screen with status filtering#1

Draft
Copilot wants to merge 3 commits into26r1from
copilot/add-release-deposits-action
Draft

Add Release action to Deposits processing screen with status filtering#1
Copilot wants to merge 3 commits into26r1from
copilot/add-release-deposits-action

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 26, 2026

Adds a bulk Release action to the existing Print/Email processing screen for CA Deposits, with a guard to only surface releasable documents when that action is selected.

Changes

  • New "Release" action — added "R" / "Release" to the ProcessingAction dropdown and wired ReleaseRecords as the processing delegate
  • ReleaseRecords handler — loads each selected deposit into CADepositEntry and calls release.Press(); per-item try/catch ensures one failure doesn't abort the batch, with a null-guard if the document isn't found
  • Status filteringRowSelecting event excludes already-released deposits (CADocStatus.Released) from the grid whenever the Release action is active; Print/Email actions are unaffected
// Hides released deposits from the grid when action == "R"
protected virtual void _(Events.RowSelecting<CADepositWithSelected> e)
{
    if (e.Row != null && FilterView.Current?.ProcessingAction == "R"
        && e.Row.Status == CADocStatus.Released)
    {
        e.Cancel = true;
    }
}

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI changed the title [WIP] Add action to release deposits on processing screen Add Release action to Deposits processing screen Mar 26, 2026
Copilot AI requested a review from dnaumov March 26, 2026 17:10
Copilot AI changed the title Add Release action to Deposits processing screen Add Release action to Deposits processing screen with status filtering Mar 26, 2026
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.

2 participants