Skip to content

Feature/ab#32350 portal payment status filter#2225

Merged
AndreGAot merged 3 commits into
devfrom
feature/AB#32350-portal-payment-status-filter
Apr 6, 2026
Merged

Feature/ab#32350 portal payment status filter#2225
AndreGAot merged 3 commits into
devfrom
feature/AB#32350-portal-payment-status-filter

Conversation

@AndreGAot
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the applicant portal “Payment Info” profile data provider to only return payments that CAS indicates are Fully Paid, aligning the portal’s payment listing with a paid-only filter requirement.

Changes:

  • Filter PaymentRequest results to only include CAS PaymentStatus == "Fully Paid" and return a constant "Fully Paid" in the DTO.
  • Update unit tests to reflect the paid-only behavior and new status string.
  • Update applicant portal documentation to describe the paid-only filtering and response shape.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
documentation/applicant-portal/applicant-profile-data-providers.md Documents paid-only filtering and the updated paymentStatus value in the example/mapping table.
applications/Unity.GrantManager/test/Unity.GrantManager.Application.Tests/Applicants/PaymentInfoDataProviderTests.cs Adjusts tests to expect "Fully Paid" and adds a paid-only filtering test.
applications/Unity.GrantManager/src/Unity.GrantManager.Application/ApplicantProfile/PaymentInfoDataProvider.cs Implements paid-only filtering and forces DTO status to "Fully Paid".

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 6, 2026

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 570
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 6, 2026

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 575
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@AndreGAot AndreGAot merged commit fb9fcaf into dev Apr 6, 2026
22 checks passed
@AndreGAot AndreGAot deleted the feature/AB#32350-portal-payment-status-filter branch April 6, 2026 19:11
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