You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So two different "approver" identities are surfaced for the same execution:
The toast names the email of the user who is hard-coded somewhere (likely the original Archera contact for the org), instead of the Admin-set notification email.
The Purchase History row correctly shows the Admin-set notification email.
This is confusing and a real bug: it is unclear who actually receives the approval email. If the toast is correct, the Admin Settings field is ignored. If the Purchase History row is correct, the toast displays stale/wrong data.
Reproduction
Set Notification email = cristi@leanercloud.com under Admin > General Settings.
From Opportunities, select an Azure recommendation -> "Purchase 1 selected" -> "Send all for approval".
Observe the toast: it names a different email than cristi@leanercloud.com.
Open Purchase History -> the row shows awaiting approval from cristi@leanercloud.com.
Fix direction
Trace the toast message's data source. The likely culprits:
Frontend hardcoding the Archera contact email when building the toast string (look for jcorbett or a default approver constant in frontend/src/).
API returning a different approver field than what Purchase History reads. The Purchase History row clearly uses the right notification email, so the toast just needs the same data source.
Whichever is wrong, fix it so both UIs show the Admin-set notification email (and the email actually goes there).
Tests required
Acceptance test: change Notification email in Admin Settings, kick off an Azure purchase approval from Opportunities, assert the toast names the same email shown in Purchase History.
Backend regression: ensure the approval-email-send path uses config.notification_email, not a hardcoded default.
Verified shipped: PR #740 (commit 67cd495, merged 2026-05-27) is in feat/multicloud-web-frontend. handler_purchases.go line 1527 now reads GlobalConfig.NotificationEmail for approval toast; git show confirms the fix. Tests added in purchase-execution-toast.test.ts and coverage_gaps_test.go.
Activity
fix(api/approvals): toast and history use Admin notification_email, n…
cristim commentedon May 28, 2026
Verified shipped: PR #740 (commit 67cd495, merged 2026-05-27) is in feat/multicloud-web-frontend. handler_purchases.go line 1527 now reads GlobalConfig.NotificationEmail for approval toast; git show confirms the fix. Tests added in purchase-execution-toast.test.ts and coverage_gaps_test.go.