Skip to content

docs(enterprise): tell self-hosted admins to upload the App logo - #12391

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/jd/jd/mrgfy-418-add-procedure-to-add-logo-manually/tell-self-hosted-admins-upload-app-logo--a11c411b
Aug 14, 2026
Merged

docs(enterprise): tell self-hosted admins to upload the App logo#12391
mergify[bot] merged 1 commit into
mainfrom
devs/jd/jd/mrgfy-418-add-procedure-to-add-logo-manually/tell-self-hosted-admins-upload-app-logo--a11c411b

Conversation

@jd

@jd jd commented Aug 14, 2026

Copy link
Copy Markdown
Member

The GitHub App the Enterprise installer creates has no logo, so Mergify's
check runs render with GitHub's default placeholder on every pull request
of every self-hosted install. Nothing in the Enterprise guide mentioned
it, so admins had no reason to know it was theirs to fix.

Add a "Set the GitHub App Logo" section to the installation guide, right
after the App is created and installed, plus a pointer to it from the
legacy manual-installation page. It covers where an org-owned App's
settings page lives on either github.com or a GHES host, the Display
information -> Upload a logo -> crop -> Set new avatar flow, the file
constraints, and the Badge background color field that only shows up once
a logo is uploaded.

The section says in one sentence why this is manual, so nobody goes
looking for an API: GitHub's App manifest rejects any image field, and an
App registration cannot be modified through the API after creation.

Ship a square asset for admins to upload: public/mergify-github-app-logo.png,
a 200x200 PNG of the Mergify mark matching the one the hosted App uses, so
self-hosted checks look like the hosted ones. The installer's vendored
wordmark is 1024x275 and crops badly, so it is not usable here.

MRGFY-418

The GitHub App the Enterprise installer creates has no logo, so Mergify's
check runs render with GitHub's default placeholder on every pull request
of every self-hosted install. Nothing in the Enterprise guide mentioned
it, so admins had no reason to know it was theirs to fix.

Add a "Set the GitHub App Logo" section to the installation guide, right
after the App is created and installed, plus a pointer to it from the
legacy manual-installation page. It covers where an org-owned App's
settings page lives on either github.com or a GHES host, the Display
information -> Upload a logo -> crop -> Set new avatar flow, the file
constraints, and the Badge background color field that only shows up once
a logo is uploaded.

The section says in one sentence why this is manual, so nobody goes
looking for an API: GitHub's App manifest rejects any image field, and an
App registration cannot be modified through the API after creation.

Ship a square asset for admins to upload: public/mergify-github-app-logo.png,
a 200x200 PNG of the Mergify mark matching the one the hosted App uses, so
self-hosted checks look like the hosted ones. The installer's vendored
wordmark is 1024x275 and crops badly, so it is not usable here.

MRGFY-418

Change-Id: Ia11c411bad8e7bab1e7ae933961e751e70099fd9
Copilot AI lite review requested due to automatic review settings August 14, 2026 09:15
@mergify
mergify Bot deployed to Mergify Merge Protections August 14, 2026 09:15 Active
@mergify

mergify Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 6 merge protections satisfied — ready to merge.

Show 6 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 👀 Review Requirements

  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • all of:
      • author = mergify-ci-bot
      • -head ~= ^docs-agent/

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 🔎 Reviews

  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds explicit documentation for Mergify Enterprise (self-hosted) administrators to upload a GitHub App logo so check runs don’t show GitHub’s default placeholder image, and cross-links the legacy manual-installation page to this new guidance.

Changes:

  • Adds a new “Set the GitHub App Logo” section to the Enterprise installation guide with step-by-step instructions and a downloadable logo link.
  • Adds a pointer from the legacy manual-installation guide to the new logo section.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/content/docs/enterprise/manual-installation-legacy.mdx Adds a short note explaining why check runs show a placeholder and links to the new logo instructions.
src/content/docs/enterprise/installation.mdx Introduces the new “Set the GitHub App Logo” section and references a downloadable logo asset.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mergify
mergify Bot requested a review from a team August 14, 2026 09:18
@jd
jd marked this pull request as ready for review August 14, 2026 09:34
@mergify
mergify Bot requested a review from a team August 14, 2026 10:23
@mergify

mergify Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 2 minutes 31 seconds in the queue, including 2 minutes 3 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Aug 14, 2026
@mergify
mergify Bot merged commit f63b09b into main Aug 14, 2026
11 checks passed
@mergify
mergify Bot deleted the devs/jd/jd/mrgfy-418-add-procedure-to-add-logo-manually/tell-self-hosted-admins-upload-app-logo--a11c411b branch August 14, 2026 11:59
@mergify mergify Bot removed the queued label Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants