Move APNs instructions out of UI and into guides

[noahtalerman]

• @noahtalerman: For the following quick win:
  • Move APNs, Apple Business, and VPP instructions out of UI and into guides #43435

Summary by CodeRabbit

• Style

  • Updated layout alignment and spacing for Apple MDM certificate setup and renewal screens.

• Refactor

  • Simplified in-app certificate setup instructions to link to external guides instead of displaying step-by-step instructions inline.
  • Updated documentation route redirect for Apple MDM renewal guidance.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.89%. Comparing base (2078e82) to head (79d3e41).
⚠️ Report is 95 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff            @@
##             main   #43434    +/-   ##
========================================
  Coverage   66.89%   66.89%            
========================================
  Files        2594     2594            
  Lines      207757   207757            
  Branches     9211     9325   +114     
========================================
  Hits       138973   138973            
  Misses      56143    56143            
  Partials    12641    12641            

Flag	Coverage Δ
frontend	54.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

_{Tip: disable this comment in your organization's Code Review settings.}

[coderabbitai]

Walkthrough

This PR simplifies the Apple MDM setup and renewal certificate workflows by removing step-by-step instructional text from two UI components and replacing it with links to an external Fleet guide. The ApplePushCertSetup.tsx and RenewCertModal.tsx components now display a single description sentence directing users to a guide instead of inline ordered instructions. The DownloadCSR and FileUploader components are repositioned within the layout, and SCSS styling is updated to reflect the new flex-based positioning and spacing requirements. Additionally, a legacy redirect URL for APNs renewal documentation is corrected to point to the appropriate guide anchor.

Possibly related issues

• Move APNs, Apple Business, and VPP instructions out of UI and into guides #43435: This PR addresses the objective to consolidate Apple Push Notification Service (APNs) certificate setup and renewal instructions by moving them out of inline UI components into external guide documentation.

Possibly related PRs

• UI housekeeping: Update Modal.children from JSX.Element to React.ReactNode, remove empty fragment wrappers #41394: Both PRs restructure the same admin MDM modal and setup component layouts, repositioning DownloadCSR and FileUploader elements outside of instruction containers and simplifying modal content organization.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description lacks required sections including checklist items, testing details, and proper issue reference format.	Complete the PR description template by adding the 'Related issue' field properly, filling out relevant checklist items, and adding Testing, Database migrations, and other applicable sections.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title accurately summarizes the main change: moving APNs instructions from the UI into guides documentation.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch move-apns-instructions

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sgress454]

Is this accurate? As of 4.84 at least, you have to click "Edit" first before you can get to "Renew certificate"

[noahtalerman]

You're right! Good catch. Opened a follow up PR: #43632