Skip to content

Conversation

@Germey
Copy link
Member

@Germey Germey commented Jan 3, 2026

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

Copilot AI review requested due to automatic review settings January 3, 2026 03:44
@Germey Germey enabled auto-merge (squash) January 3, 2026 03:44
@Germey Germey changed the title feature: update logo fix: update logo Jan 3, 2026
@Germey Germey changed the title fix: update logo feat: update logo Jan 3, 2026
Copy link

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

This PR adds support for displaying an Ace badge overlay on the Dify logo component. The change introduces a new optional parameter to control badge visibility and adds styling/positioning maps for the badge at different logo sizes.

Key changes:

  • Added new logo asset (acedatacloud-logo.ico)
  • Enhanced DifyLogo component with optional Ace badge overlay functionality
  • Implemented size-responsive positioning for the badge

Reviewed changes

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

File Description
web/public/logo/acedatacloud-logo.ico New binary logo asset added for Ace branding
web/app/components/base/logo/dify-logo.tsx Modified logo component to support optional Ace badge overlay with responsive sizing and positioning

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Germey Germey merged commit 23cd0d4 into main Jan 3, 2026
19 of 20 checks passed
@Germey Germey deleted the update-logo branch January 3, 2026 04:15
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