Skip to content

Remove Anthropic privacy notice from Expense Agent - #10341

Merged
encimita merged 9 commits into
mainfrom
encimita-remove-anthropic-expense-agent
Aug 19, 2026
Merged

Remove Anthropic privacy notice from Expense Agent#10341
encimita merged 9 commits into
mainfrom
encimita-remove-anthropic-expense-agent

Conversation

@encimita

@encimita encimita commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Fixes AB#646070

Summary

  • Remove Anthropic registration, approval checks, capability requirements, and active privacy notice UI from Expense Agent.
  • Keep the original page 7078 implementation unchanged, mark it pending obsolete, and wrap it in #if not CLEAN29.
  • Attempt to remove the persisted legacy privacy notice and its approval records during the per-database upgrade without blocking the upgrade.
  • Keep the legacy notice ID only as migration data so existing records can be deleted.

Testing

  • No tests added or changed; no Expense Agent tests reference Anthropic.
  • Local AL compilation could not run because NAV initialization requires registry permissions unavailable in this worktree session.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
@github-actions github-actions Bot added AL: Apps (W1) Add-on apps for W1 Integration GitHub request for Integration area labels Aug 18, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f556efb0-534d-48fa-8818-be0d8ca8579d
@encimita
encimita marked this pull request as ready for review August 18, 2026 09:40
@encimita
encimita requested a review from a team August 18, 2026 09:40
@github-actions github-actions Bot added this to the Version 29.0 milestone Aug 18, 2026
@encimita encimita removed the Integration GitHub request for Integration area label Aug 18, 2026
@github-actions github-actions Bot added the Integration GitHub request for Integration area label Aug 18, 2026
@onbuyuka Onat Buyukakkus (onbuyuka) added Ownership: Manual Preserve the manually selected team ownership and removed Integration GitHub request for Integration area labels Aug 18, 2026
@github-actions github-actions Bot added the Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction label Aug 18, 2026
@onbuyuka Onat Buyukakkus (onbuyuka) added the Other GitHub request for other area than SCM, Finance or Integration label Aug 18, 2026
@github-actions github-actions Bot removed the Ownership: Needs Review Ownership is Other, low confidence, or needs manual correction label Aug 18, 2026
@encimita
encimita added this pull request to the merge queue Aug 19, 2026
Merged via the queue into main with commit c043093 Aug 19, 2026
337 of 344 checks passed
@encimita
encimita deleted the encimita-remove-anthropic-expense-agent branch August 19, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 Other GitHub request for other area than SCM, Finance or Integration Ownership: Manual Preserve the manually selected team ownership

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants