Skip to content

Remove GitHub AE references from Enterprise login screen#21926

Merged
sergiou87 merged 3 commits into
desktop:developmentfrom
toddmanion:fix/remove-ae-references
Apr 9, 2026
Merged

Remove GitHub AE references from Enterprise login screen#21926
sergiou87 merged 3 commits into
desktop:developmentfrom
toddmanion:fix/remove-ae-references

Conversation

@toddmanion
Copy link
Copy Markdown
Contributor

Description

GitHub AE has been discontinued. This PR updates the Enterprise sign-in UI to remove all references to it.

Changes

  • Label: Enterprise or AE addressEnterprise address
  • Placeholder URL: https://github.example.comhttps://example.ghe.com
  • Error message: Updated the invalid URL example to use https://example.ghe.com
  • Test: Updated to match the new label text

Fixes #21925

GitHub AE has been discontinued. This updates the Enterprise sign-in
UI to:
- Change label from 'Enterprise or AE address' to 'Enterprise address'
- Update placeholder URL from https://github.example.com to
  https://example.ghe.com
- Update the invalid URL error message with the new example URL
- Update corresponding test

Fixes desktop#21925

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@toddmanion toddmanion requested a review from a team as a code owner April 8, 2026 21:23
@toddmanion toddmanion requested review from Copilot and niik April 8, 2026 21:23
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 8, 2026

Thanks for your pull request! Unfortunately, it doesn't meet the minimum requirements for review:

  • None of the referenced issues have the help wanted label

Please update your PR to address the above. Requirements:

  1. Include a detailed description of what this PR does
  2. Link to an issue with the help wanted label (use Fixes #123 or Closes #123 if it resolves the issue)

This PR will be automatically closed in 7 days if these requirements are not met.

@github-actions github-actions Bot added external Pull request originating outside of the Desktop core team needs-triage labels Apr 8, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates GitHub Desktop’s Enterprise sign-in UI copy to remove references to GitHub AE (now discontinued) and align the example Enterprise address shown to users.

Changes:

  • Renames the Enterprise endpoint label to “Enterprise address” where applicable.
  • Updates the example/placeholder Enterprise URL to https://example.ghe.com in the Enterprise sign-in UI.
  • Updates the invalid-URL error message example and adjusts a unit test to match the new label.
Show a summary per file
File Description
app/test/unit/ui/welcome-and-sign-in-wrappers-test.tsx Updates the Enterprise address label used by the UI test query.
app/src/ui/sign-in/sign-in.tsx Updates the Enterprise endpoint placeholder to the new example URL.
app/src/ui/lib/enterprise-server-entry.tsx Updates the Enterprise address label and placeholder in the Enterprise entry form component.
app/src/lib/stores/sign-in-store.ts Updates the invalid URL error message example to the new placeholder URL.

Copilot's findings

  • Files reviewed: 4/4 changed files
  • Comments generated: 0

toddmanion and others added 2 commits April 8, 2026 14:41
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@sergiou87 sergiou87 merged commit cb5bc0b into desktop:development Apr 9, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

external Pull request originating outside of the Desktop core team needs-triage unmet-requirements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enterprise login screen has a mention to AE

3 participants