Skip to content

bugfix/AB#32218 convert to as single query explicit for attachment app service get#2481

Merged
JamesPasta merged 1 commit into
devfrom
bugfix/AB#32218-intermittent-failed-err-chunks
May 22, 2026
Merged

bugfix/AB#32218 convert to as single query explicit for attachment app service get#2481
JamesPasta merged 1 commit into
devfrom
bugfix/AB#32218-intermittent-failed-err-chunks

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

This PR updates the GrantManager attachment retrieval query to execute asynchronously via EF Core and explicitly enforce single-query execution.

Changes:

  • Add EF Core extensions import (Microsoft.EntityFrameworkCore) to support async query execution.
  • Materialize the attachments/person join with AsSingleQuery().ToListAsync() instead of synchronous ToList().

@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

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

📄 HTML Reports

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

@AndreGAot AndreGAot changed the title AB#32218 convert to as single query explicit for app attachments AB#32218 convert to as single query explicit for attachment app service get May 22, 2026
@AndreGAot AndreGAot changed the title AB#32218 convert to as single query explicit for attachment app service get bugfix/AB#32218 convert to as single query explicit for attachment app service get May 22, 2026
@JamesPasta JamesPasta merged commit ef2c95b into dev May 22, 2026
26 checks passed
@JamesPasta JamesPasta deleted the bugfix/AB#32218-intermittent-failed-err-chunks branch May 22, 2026 21:12
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.

3 participants