Skip to content

docs: use stable Amplitude logo URL - #82

Merged
aliaksandr-kazarez merged 1 commit into
mainfrom
codex/sdk-353-stable-logo
Oct 8, 2026
Merged

aliaksandr-kazarez merged 1 commit into
mainfrom
codex/sdk-353-stable-logo

Conversation

@aliaksandr-kazarez

@aliaksandr-kazarez aliaksandr-kazarez commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Replace the README logo URL tied to an old frontend build with the stable Amplitude brand URL. This prevents the logo from depending on retention of that build asset.

Related: SDK-353.

Validation

  • Verified that the diff contains only exact logo URL substitutions and no old URL remains in tracked files.
  • git diff --check passed.
  • Runtime tests not run: documentation-only URL change.

Checklist

  • Does your PR title have the correct title format?
  • Does your PR have a breaking change?: No; no runtime or public API changes.

Prepared with Codex. LLMs make mistakes — please verify before acting.


Note

Low Risk
Documentation-only README image URL change with no runtime or API impact.

Overview
Updates the README header image to load the Amplitude logo from the stable https://static.amplitude.com/brand/amplitude-logo-with-text.svg URL instead of a lightning/... build-specific asset path, so the logo does not break if that old frontend build is removed.

Reviewed by Cursor Bugbot for commit c984541. Bugbot is set up for automated code reviews on this repo. Configure here.

@linear-code

linear-code Bot commented Oct 8, 2026

Copy link
Copy Markdown

SDK-353

@aliaksandr-kazarez
aliaksandr-kazarez marked this pull request as ready for review October 8, 2026 17:15
@aliaksandr-kazarez
aliaksandr-kazarez requested a review from a team as a code owner October 8, 2026 17:15
@aliaksandr-kazarez
aliaksandr-kazarez merged commit 77cf5b7 into main Oct 8, 2026
8 checks passed
@aliaksandr-kazarez
aliaksandr-kazarez deleted the codex/sdk-353-stable-logo branch October 8, 2026 18:42

This branch was previously deployed

1 inactive deployment
Unit Test — c9845411 Deployed Oct 8, 2026 by aliaksandr-kazarez via test (3.14) #226
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.

3 participants