Skip to content

Conversation

@LevisNgigi
Copy link
Contributor

fix(logo): fix logo url typo

SUMMARY

This PR corrects a typo in the logo URL that was causing an incorrect redirect. The updated URL now points to the intended location,the Apache Superset home page,ensuring consistent and expected navigation when the logo is clicked.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

  1. Open the Superset home page (or any page where the logo appears, e.g., the top navigation bar).

  2. Click the logo.

  3. Confirm that it now correctly navigates to the Apache Superset home page

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

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

I've completed my review and didn't find any issues.

Files scanned
File Path Reviewed
superset-frontend/packages/superset-ui-core/src/theme/Theme.tsx

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@LevisNgigi LevisNgigi closed this Jun 23, 2025
@LevisNgigi LevisNgigi reopened this Jun 23, 2025
@hainenber
Copy link
Contributor

LGTM. Thanks!

@hainenber hainenber merged commit d3b781b into apache:master Jun 23, 2025
124 of 126 checks passed
amaannawab923 added a commit to amaannawab923/superset that referenced this pull request Jun 24, 2025
* fix(logo): fix logo url typo (apache#33867)

* chore(Icons): Add UsergroupAddOutlined icon (apache#33866)

* fix(api): Added uuid as a valid search column (apache#33833)

* fix(native filters): Make the Apply button available after click on Clear All (apache#33834)

* build(deps): remove legacy browser polyfills (apache#33853)

Signed-off-by: hainenber <[email protected]>

* build(dev-deps): update `fetch-mock` to v11 (apache#33854)

Signed-off-by: hainenber <[email protected]>

* build(be/deps): replace `importlib_metadata` usage with native  Python 3.10+ `importlib.metadata`  (apache#33868)

Signed-off-by: hainenber <[email protected]>

* chore: Updates files related to 5.0.0 release (apache#33869)

* chore(docs): bump references to docker image versions upon release of 5.0.0 (apache#33872)

---------

Signed-off-by: hainenber <[email protected]>
Co-authored-by: Levis Mbote <[email protected]>
Co-authored-by: Enzo Martellucci <[email protected]>
Co-authored-by: Paul Rhodes <[email protected]>
Co-authored-by: Vitor Avila <[email protected]>
Co-authored-by: Đỗ Trọng Hải <[email protected]>
Co-authored-by: Michael S. Molina <[email protected]>
Co-authored-by: Sam Firke <[email protected]>
LisaHusband pushed a commit to LisaHusband/superset that referenced this pull request Jun 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants