Skip to content

Replace Blazor WASM page load with direct API endpoint for error acknowledgment#48

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/fix-acknowledge-link-loading
Draft

Replace Blazor WASM page load with direct API endpoint for error acknowledgment#48
Copilot wants to merge 4 commits intomainfrom
copilot/fix-acknowledge-link-loading

Conversation

Copy link
Contributor

Copilot AI commented Nov 22, 2025

Error acknowledgment links in email notifications currently point to a Blazor WASM page that loads the entire web application to make a single API call, causing slow load times.

Changes

API Layer

  • Added GET /Jobs/AcknowledgeError/{acknowledgementCode:guid} endpoint that returns static HTML
  • Endpoint calls existing AcknowledgeErrorAsync method, returns styled success/error page
  • Marked [AllowAnonymous] for public access without authentication
  • Logs exceptions server-side, displays generic error messages to users
  • Kept existing POST endpoint for Blazor WASM backward compatibility

Email Link Generation

  • ServiceChecker: Changed from {AppUrl}acknowledge-error/{code} to {ApiUrl}/Jobs/AcknowledgeError/{code}
  • JobManager: Updated to use API endpoint format instead of app route

Example

Email links now go directly to API:

Before: https://app.example.com/acknowledge-error/abc123
After:  https://api.example.com/Jobs/AcknowledgeError/abc123

The GET endpoint returns a simple HTML page:

<div class="container">
  <div class="icon success-icon"></div>
  <h1>Error Acknowledged</h1>
  <p>The error has been successfully acknowledged.</p>
</div>

Database validation (job exists, not already acknowledged) remains unchanged in stored procedure [app].[Jobs_AcknowledgeError].

Original prompt

This section details on the original issue you should resolve

<issue_title>Acknowledge link should not load full app</issue_title>
<issue_description>Takes forever since it has to load the web app. Should be a link directly to the API call for acknowledging, and return a static message rather than a link to the public site</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits November 22, 2025 17:48
Co-authored-by: jtenos <607782+jtenos@users.noreply.github.com>
Co-authored-by: jtenos <607782+jtenos@users.noreply.github.com>
Co-authored-by: jtenos <607782+jtenos@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix acknowledge link to directly call API instead of loading app Replace Blazor WASM page load with direct API endpoint for error acknowledgment Nov 22, 2025
Copilot AI requested a review from jtenos November 22, 2025 17:57
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.

Acknowledge link should not load full app

2 participants