Skip to content

fix(web): open embedded recommend banner plugins in the detail dialog - #41828

Merged
CodingOnStar merged 686 commits into
mainfrom
feat/creator-profile-home
Sep 7, 2026
Merged

CodingOnStar merged 686 commits into
mainfrom
feat/creator-profile-home

Conversation

@CodingOnStar

@CodingOnStar CodingOnStar commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

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

Follow-up to #41538. In the in-app marketplace, clicking a plugin on a recommend banner now opens the same MarketplaceDetailDialog as homepage plugin cards. The standalone marketplace still navigates to the plugin detail page.

Related marketplace submodule bump: langgenius/dify-marketplace#163.

Screenshots

N/A — click-target change covered by unit tests.

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 make lint && make type-check (backend) and vp staged (frontend) to appease the lint gods

fatelei and others added 30 commits August 11, 2026 07:24
# Conflicts:
#	oxlint-suppressions.json
#	packages/contracts/marketplace.ts
#	web/app/(commonLayout)/marketplace/page.tsx
#	web/app/components/main-nav/routes.ts
#	web/app/components/plugins/marketplace/detail-dialog/__tests__/index.spec.tsx
#	web/app/components/plugins/marketplace/detail-dialog/index.tsx
#	web/app/components/plugins/marketplace/home/__tests__/home-catalog-navigation.spec.tsx
#	web/app/components/plugins/marketplace/home/__tests__/home-header.spec.tsx
#	web/app/components/plugins/marketplace/home/__tests__/home-trending.spec.tsx
#	web/app/components/plugins/marketplace/home/home-catalog-tabs.tsx
#	web/app/components/plugins/marketplace/home/home-header.tsx
#	web/app/components/plugins/marketplace/home/home-search.tsx
#	web/app/components/plugins/marketplace/home/home-trending.tsx
#	web/app/components/plugins/marketplace/home/index.tsx
#	web/app/components/plugins/marketplace/index.tsx
#	web/app/components/plugins/marketplace/list/__tests__/card-wrapper.spec.tsx
#	web/app/components/plugins/marketplace/list/list-wrapper.tsx
#	web/app/components/plugins/marketplace/view.tsx
CodingOnStar added 9 commits September 4, 2026 15:25
Independent Marketplace does not load the tools namespace, so
tools.author fell back to the key "author". Use plugin.marketplace.by.
Keep marketplace.creatorProfile.loadMore and loadMoreFailed from
falling back to English in non-en locales. Dutch by-author copy is Door.
…home

# Conflicts:
#	web/app/components/plugins/marketplace/list/__tests__/list-with-collection.spec.tsx
#	web/app/components/plugins/marketplace/list/carousel.tsx
#	web/app/components/plugins/marketplace/list/list-with-collection.tsx
CI Style Check / TS Common failed on vp check formatting for this file.
Console Docker keeps MARKETPLACE_API_URL as the marketplace origin and
entrypoint derives NEXT_PUBLIC_MARKETPLACE_API_PREFIX with /api/v1.
SSR preferred the bare origin, so /templates hit HTML instead of the API
and showed marketplace.loadError on staging.
# Conflicts:
#	web/__tests__/proxy-frame-options.spec.ts
#	web/config/index.ts
#	web/i18n/ar-TN/plugin.json
#	web/i18n/de-DE/common.json
#	web/i18n/de-DE/plugin.json
#	web/i18n/es-ES/plugin.json
#	web/i18n/fa-IR/plugin.json
#	web/i18n/fr-FR/plugin.json
#	web/i18n/hi-IN/plugin.json
#	web/i18n/id-ID/plugin.json
#	web/i18n/it-IT/plugin.json
#	web/i18n/ko-KR/plugin.json
#	web/i18n/lo-LA/plugin.json
#	web/i18n/nl-NL/plugin.json
#	web/i18n/pl-PL/plugin.json
#	web/i18n/ro-RO/plugin.json
#	web/i18n/ru-RU/plugin.json
#	web/i18n/sl-SI/plugin.json
#	web/i18n/th-TH/plugin.json
#	web/i18n/tr-TR/plugin.json
#	web/i18n/uk-UA/plugin.json
#	web/i18n/vi-VN/plugin.json
In-app marketplace recommend cards now reuse the plugin tab dialog instead of navigating away. The standalone marketplace still routes to the plugin page.
@github-actions github-actions Bot added the web This relates to changes on the web. label Sep 4, 2026
@codecov

codecov Bot commented Sep 5, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 87.14%. Comparing base (d93e5aa) to head (fa0e135).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
.../plugins/marketplace/home/home-trending-slides.tsx 98.03% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main   #41828   +/-   ##
=======================================
  Coverage   87.13%   87.14%           
=======================================
  Files        5413     5413           
  Lines      311849   311899   +50     
  Branches    62656    62652    -4     
=======================================
+ Hits       271732   271790   +58     
+ Misses      34806    34798    -8     
  Partials     5311     5311           
Flag Coverage Δ
dify-ui 93.74% <ø> (ø)
web 87.35% <98.33%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@CodingOnStar
CodingOnStar added this pull request to the merge queue Sep 7, 2026
Merged via the queue into main with commit 762dc5e Sep 7, 2026
51 checks passed
@CodingOnStar
CodingOnStar deleted the feat/creator-profile-home branch September 7, 2026 03:27
CodingOnStar added a commit that referenced this pull request Sep 7, 2026
…#41828)

Co-authored-by: fatelei <fatelei@gmail.com>
Co-authored-by: zxhlyh <jasonapring2015@outlook.com>
Co-authored-by: CodingOnStar <hanxujiang@dify.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: 姜涵煦 <hanxujiang@jianghanxudeMacBook-Pro-2.local>
Co-authored-by: L1nSn0w <l1nsn0w@qq.com>
Co-authored-by: yyh <92089059+lyzno1@users.noreply.github.com>

(cherry picked from commit 762dc5e)
CodingOnStar added a commit that referenced this pull request Sep 7, 2026
…#41828)

Co-authored-by: fatelei <fatelei@gmail.com>
Co-authored-by: zxhlyh <jasonapring2015@outlook.com>
Co-authored-by: CodingOnStar <hanxujiang@dify.com>
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: 姜涵煦 <hanxujiang@jianghanxudeMacBook-Pro-2.local>
Co-authored-by: L1nSn0w <l1nsn0w@qq.com>
Co-authored-by: yyh <92089059+lyzno1@users.noreply.github.com>

(cherry picked from commit 762dc5e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants