Skip to content

Conversation

@boris-w
Copy link
Contributor

@boris-w boris-w commented Dec 23, 2025

Related Issues: T1389

@boris-w boris-w changed the title feat: add template related in detail feat: add template related in detail (T1389) Dec 23, 2025
@boris-w boris-w requested a review from Copilot December 23, 2025 04:15
Copy link
Contributor

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 a "related templates" feature to the template detail view, allowing users to discover and navigate to similar templates while viewing template details. The implementation includes new i18n translations, a new RecommendTemplate component, and updates to the template navigation flow.

Key changes:

  • Introduces a RecommendTemplate component that displays 3 featured templates at the bottom of template detail pages
  • Adds template-to-template navigation capability via onTemplateClick callback prop
  • Improves TemplatePreview loading state handling with URL-based effect triggers
  • Implements smooth scroll-to-top behavior when switching between templates

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
packages/common-i18n/src/locales/en/common.json Adds English translations for "Related Templates" and "No image"
packages/common-i18n/src/locales/zh/common.json Adds Chinese translations for related templates feature
apps/nestjs-backend/src/types/i18n.generated.ts Updates generated TypeScript types for new translation keys
apps/nextjs-app/src/features/app/components/space/template/RecommendTemplate.tsx New component that fetches and displays 3 featured templates with click handling
apps/nextjs-app/src/features/app/components/space/template/TemplateDetail.tsx Integrates RecommendTemplate component and adds scroll-to-top behavior with refs
apps/nextjs-app/src/features/app/components/space/template/TemplatePreview.tsx Refactors URL computation and adds useEffect to reset loading state on URL changes
apps/nextjs-app/src/features/app/components/space/template/TemplateSheet.tsx Passes onTemplateClick callback to TemplateDetail for navigation
apps/nextjs-app/src/features/app/components/space/template/TemplateModal.tsx Adds animated width transition and passes onTemplateClick callback

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

@boris-w boris-w merged commit 1b990f9 into develop Dec 23, 2025
18 of 19 checks passed
@boris-w boris-w deleted the feat/template-related branch December 23, 2025 04:22
@github-actions
Copy link

🧹 Preview Environment Cleanup

@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 20451049725

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 71.827%

Totals Coverage Status
Change from base Build 20438648003: 0.002%
Covered Lines: 59623
Relevant Lines: 83009

💛 - Coveralls

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.

3 participants