Skip to content

Conversation

@Innei
Copy link
Contributor

@Innei Innei commented Jun 23, 2025

  • chore(ssr): update global constants and clean up imports
  • feat: redeisgn og
  • refactor(desktop): rename and restructure navigation functions for clarity
  • feat(desktop): add debug logging in EntryItemWrapper for click handling
  • refactor(desktop): remove debug logging from handleClick in EntryItemWrapper
  • cleanup

Innei added 4 commits June 23, 2025 18:51
…arity

- Renamed `navigateEntry` to `getNavigateEntryPath` for better clarity on its purpose.
- Introduced a memoized `navigationPath` in `EntryItemWrapper` to optimize path generation.
- Updated the navigation logic to utilize the new function, improving code organization and readability.

Signed-off-by: Innei <tukon479@gmail.com>
- Introduced console logging in the handleClick function to track navigation decisions and entry details.
- This addition aids in debugging navigation logic by providing insights into the entryId and feedId during click events.

Signed-off-by: Innei <tukon479@gmail.com>
…Wrapper

- Commented out console logging in the handleClick function to clean up the code and reduce unnecessary output during click events.
- This change maintains the functionality while improving the overall readability of the code.

Signed-off-by: Innei <tukon479@gmail.com>
Signed-off-by: Innei <tukon479@gmail.com>
@vercel
Copy link

vercel bot commented Jun 23, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
follow ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 23, 2025 11:38am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
follow-external-ssr ⬜️ Ignored (Inspect) Visit Preview Jun 23, 2025 11:38am

@Innei Innei changed the base branch from dev to main June 23, 2025 11:35
@Innei Innei merged commit d6d9044 into main Jun 23, 2025
8 checks passed
@Innei Innei deleted the test/link branch June 23, 2025 11:39
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