Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BACKLOG] Prevent Content Layout Shift to improve SEO #472

Open
1 of 6 tasks
saenyakorn opened this issue Jan 29, 2023 · 0 comments
Open
1 of 6 tasks

[BACKLOG] Prevent Content Layout Shift to improve SEO #472

saenyakorn opened this issue Jan 29, 2023 · 0 comments
Labels
cugetreg-api Related to cugetreg-api

Comments

@saenyakorn
Copy link
Member

Problem

On our Course Search, it'll take 3 steps to render the page.

  1. Load everything except Course Card

Screenshot 2566-01-29 at 23 58 17

  1. Fetch Course Card data and render the card

Screenshot 2566-01-29 at 23 58 27

  1. Fetch recommendation data and render purple section.

Screenshot 2566-01-29 at 23 58 37

CLS (Content Layout Shift) causes much lower overall SEO score, that affect to Google search index; other web may be indexed before ours. You can find more detail about CLS here.

It would be great if we can reduce these 3 steps into 1 step. But I'm concerning about how expensive when rendering course card from server-side (could discuss later). But, I guess we can reduce from step 2 to step 3 by caching recommendation data in Redis with small expiration time and render them from server-side.

Task Description

Implement cache from recommendation system API using Redis or something else you want.

Additional Context

No response

Related Teams

  • Frontend
  • Backend
  • Data
  • Design
  • Infra
  • QA

Task Advisors

@sae

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cugetreg-api Related to cugetreg-api
Projects
Status: Todo
Development

No branches or pull requests

1 participant