Skip to content

Add footer and privacy notice; fix Agent 365 checklist and advisory l… - #578

Merged
Alejandro Lopez (soyalejolopez) merged 1 commit into
masterfrom
alejanl-microsoft-agent-365-launch-fixes
Oct 30, 2026
Merged

Alejandro Lopez (soyalejolopez) merged 1 commit into
masterfrom
alejanl-microsoft-agent-365-launch-fixes

Conversation

@soyalejolopez

Copy link
Copy Markdown
Member

…aunch issues

  • Footer (Microsoft | FastTrack, as-is disclaimer, Terms and Conditions, Privacy Statement) and an informational "Privacy notice" modal on the catalog, every resource and download page, and the Agents cost calculator. The notice makes no consent call and records no choice.
  • Enablement Checklist 1.0.3: replace the 404 Learn link on Gateway & compute task 10, point the two Defender tasks at the current Connected platforms address, show the catalog version in the app, and send feedback to the resource feedback form. A check keeps the in-app version equal to the README version.
  • Technical Advisory 1.0.2: add a page icon so browsers stop requesting a missing favicon.ico.
  • Agents Cost Calculator 2.0.1: footer links and privacy notice only.
  • Regenerated catalog.json, sitemap.xml, feed.xml and r/**.

…aunch issues

- Footer (Microsoft | FastTrack, as-is disclaimer, Terms and Conditions, Privacy Statement) and an
  informational "Privacy notice" modal on the catalog, every resource and download page, and the
  Agents cost calculator. The notice makes no consent call and records no choice.
- Enablement Checklist 1.0.3: replace the 404 Learn link on Gateway & compute task 10, point the two
  Defender tasks at the current Connected platforms address, show the catalog version in the app,
  and send feedback to the resource feedback form. A check keeps the in-app version equal to the
  README version.
- Technical Advisory 1.0.2: add a page icon so browsers stop requesting a missing favicon.ico.
- Agents Cost Calculator 2.0.1: footer links and privacy notice only.
- Regenerated catalog.json, sitemap.xml, feed.xml and r/**.

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

Copy link
Copy Markdown
Contributor

🛰️ PR Sweeper report

Risk: 🟠 MEDIUM · Security gate: ✅ passing · Files: 100

🔒 Automated guardrails (authoritative)

Category Location Finding
⚠️ scope catalog.json catalog.json is generated by tools/catalog-build and must not be edited by hand.
⚠️ scope — Touches security-sensitive path(s): catalog.json. Maintainer + security review required.
ℹ️ scope — Change(s) outside known content roots: assets/privacy-notice.js, assets/site-footer.css, feed.xml, index.html, r/add-engagegroupadmins/download/index.html, r/add-engagegroupadmins/index.html, r/agent-365-enablement-checklist/download/index.html, r/agent-365-enablement-checklist/index.html, r/agent-365-lifecycle-atlas/index.html, r/agent-brainstorming-template/download/index.html ….
ℹ️ scope — Large PR: 100 files changed. Smaller PRs are easier to review.

🤖 Dual-model AI review (advisory)

AI review unavailable for this run (models not reachable or no diff). Guardrails above are unaffected.

  • GPT-class reviewer (openai/gpt-5): skipped — request failed: Unexpected token 'O', "OK
    " is not valid JSON
  • Deep-reasoning reviewer (openai/o3): skipped — request failed: Unexpected token 'O', "OK
    " is not valid JSON

The automated guardrails are authoritative and gate the security status. The AI review is advisory and never auto-merges. Thanks for contributing to FastTrack! 🛩️

@github-actions github-actions Bot added sweeper:risk-medium PR Sweeper: medium risk sweeper:scope-review PR Sweeper: scope/sensitive-path review labels Oct 30, 2026
'the cost calculator': [costToolSource, '../../assets/']
};
for (const [name, [source, assets]] of Object.entries(footerPages)) {
assert(source.includes('https://www.microsoft.com/en-us/legal/terms-of-use'), `${name} must link the Microsoft Terms and Conditions.`);
};
for (const [name, [source, assets]] of Object.entries(footerPages)) {
assert(source.includes('https://www.microsoft.com/en-us/legal/terms-of-use'), `${name} must link the Microsoft Terms and Conditions.`);
assert(source.includes('https://privacy.microsoft.com/en-us/privacystatement'), `${name} must link the Microsoft Privacy Statement.`);
@soyalejolopez
Alejandro Lopez (soyalejolopez) merged commit a95151a into master Oct 30, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweeper:risk-medium PR Sweeper: medium risk sweeper:scope-review PR Sweeper: scope/sensitive-path review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants