Skip to content

feat(asset-library): add search recommendations for asset types#3012

Merged
nighca merged 1 commit intogoplus:devfrom
nighca:aigc-details
Apr 9, 2026
Merged

feat(asset-library): add search recommendations for asset types#3012
nighca merged 1 commit intogoplus:devfrom
nighca:aigc-details

Conversation

@nighca
Copy link
Copy Markdown
Collaborator

@nighca nighca commented Apr 8, 2026

  • Replace search keyword on recommendation click instead of concatenating.
  • Improve recommendation items for better search results.

{ en: 'UI', zh: '界面' }
],
[AssetType.Sound]: [
// TODO: discussion needed
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

目前声音的搜索效果不好,没有合适的 recommendation 项

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request replaces the getAssetCategories utility with a static searchRecommendations object to provide search suggestions in the asset library. The review identifies a critical issue where AssetType and LocaleMessage are used in a new script block without being imported, which will cause reference errors. Additionally, it is recommended to use unique keys instead of array indices in the v-for loop to adhere to Vue best practices.

@nighca nighca marked this pull request as ready for review April 8, 2026 09:40
Copilot AI review requested due to automatic review settings April 8, 2026 09:40
Copy link
Copy Markdown
Contributor

@xgopilot xgopilot bot left a comment

Choose a reason for hiding this comment

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

One noteworthy finding and one documentation suggestion below.

Copy link
Copy Markdown
Collaborator

@cn0809 cn0809 left a comment

Choose a reason for hiding this comment

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

LGTM

@nighca nighca merged commit 8a27957 into goplus:dev Apr 9, 2026
8 checks passed
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