Skip to content

Restore missing favicon and Apple touch icon URLs (docs) - #1371

Merged
gantoreno merged 2 commits into
masterfrom
gro-1070-restore-missing-favicon-and-apple-touch-icon-urls
Oct 5, 2026
Merged

gantoreno merged 2 commits into
masterfrom
gro-1070-restore-missing-favicon-and-apple-touch-icon-urls

Conversation

@gantoreno

@gantoreno gantoreno commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Fixes GRO-1070

  • Restores missing SVGs
  • Points to the right URLs

Related: https://github.com/metabase/metabase.github.io/pull/6628

@gantoreno
gantoreno requested a review from a team October 5, 2026 16:12
@linear

linear Bot commented Oct 5, 2026

Copy link
Copy Markdown

GRO-1070

@gantoreno gantoreno changed the title fix: restore missing favicon and apple-touch-icon Restore missing favicon and Apple touch icon URLs (docs) Oct 5, 2026
@github-actions

github-actions Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

The deployments were removed when this pull request closed. Reopening it deploys again at the same address.

Comment thread _includes/head.html

{% if jekyll.environment == "development" %}
<link href="/images/favicon-dev.svg" rel="shortcut icon" />
<link href="/docs/images/favicon-dev.svg" rel="shortcut icon" />

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The 4 images already existed in public/images so they should be removed from there.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@gantoreno
gantoreno requested a review from bpander October 5, 2026 19:13

@bpander bpander left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👌

@gantoreno
gantoreno merged commit d935d7a into master Oct 5, 2026
6 checks passed
@gantoreno
gantoreno deleted the gro-1070-restore-missing-favicon-and-apple-touch-icon-urls branch October 5, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants