[PR #5227] fix: initialize site with customized icon and icon_background #24831

Closed
opened 2026-02-21 20:23:46 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/5227

State: closed
Merged: Yes


Description

This PR ensures that the icon settings specified when creating the app are also reflected in its web app.

Related #5180

Type of Change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Create new app with default icon and default icon background, then publish the app. Ensure that the web app has default icon and there is no side effect of this PR.
  • Create new app with customized icon and icon background, then publish the app. Ensure that the web app has the same icon and background as the bot.

Suggested Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods
  • optional I have made corresponding changes to the documentation
  • optional I have added tests that prove my fix is effective or that my feature works
  • optional New and existing unit tests pass locally with my changes
**Original Pull Request:** https://github.com/langgenius/dify/pull/5227 **State:** closed **Merged:** Yes --- # Description This PR ensures that the icon settings specified when creating the app are also reflected in its web app. Related #5180 ## Type of Change Please delete options that are not relevant. - [x] Bug fix (non-breaking change which fixes an issue) # How Has This Been Tested? Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration - [x] Create new app with default icon and default icon background, then publish the app. Ensure that the web app has default icon and there is no side effect of this PR. - [x] Create new app with customized icon and icon background, then publish the app. Ensure that the web app has the same icon and background as the bot. # Suggested Checklist: - [x] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [x] My changes generate no new warnings - [x] I ran `dev/reformat`(backend) and `cd web && npx lint-staged`(frontend) to appease the lint gods - [ ] `optional` I have made corresponding changes to the documentation - [ ] `optional` I have added tests that prove my fix is effective or that my feature works - [ ] `optional` New and existing unit tests pass locally with my changes
yindo added the pull-request label 2026-02-21 20:23:46 -05:00
yindo closed this issue 2026-02-21 20:23:46 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#24831