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

[#220] 시맨틱 태그 + MetaData 수정 #221

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

oauch
Copy link
Member

@oauch oauch commented Apr 2, 2024

✅ 작업 내용

  • 시맨틱 태그 추가
  • MetaData 변경

📍 리뷰 포인트

X

👀 기타 사항

  • 시멘틱 태그는 일단 마이 대시보드 페이지만 수정했습니다.
  • 이건 아마 점점 바꿔가야 될꺼 같습니다 (<- 대작업 에상)
  • app router에서는 메타 데이터를 다음과 같이 적용했었는데, page router는 이게 맞는지 자세히 모르겠네요..

SEO 점수 향상

  • 퍼포먼스를 줘버렸지만, SEO가 올라갔습니다!

추가 전
스크린샷 2024-04-03 18 09 22

추가 후
스크린샷 2024-04-04 16 55 40

Copy link

vercel bot commented Apr 2, 2024

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

Name Status Preview Comments Updated (UTC)
taskify ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 13, 2024 5:19am

@oauch oauch self-assigned this Apr 2, 2024
@oauch oauch added the ♻️ Refactor 코드 리팩토링 label Apr 2, 2024
@qooktree1
Copy link
Contributor

고생하셨습니다!!

각 페이지별로 Head 태그들도 없앨 수 있는 부분들은 삭제해도 좋을 것 같습니다

@oauch
Copy link
Member Author

oauch commented Apr 3, 2024

고생하셨습니다!!

각 페이지별로 Head 태그들도 없앨 수 있는 부분들은 삭제해도 좋을 것 같습니다

넵 참고하겠습니당

@oauch oauch linked an issue Apr 3, 2024 that may be closed by this pull request
2 tasks
@oauch oauch added this to the ♻️ Code Refactoring milestone Apr 3, 2024
Copy link
Contributor

@zermzerm zermzerm left a comment

Choose a reason for hiding this comment

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

캬 수고많으셨습니다!

@@ -75,7 +75,7 @@ const StyledDashboardGroupContainer = styled.div`
}
`;

const StyledTitleWrapper = styled.div`
const StyledTitleWrapper = styled.p`
Copy link
Contributor

Choose a reason for hiding this comment

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

div를 p로 바꾸는것도 도움이 되나요?

Copy link
Member Author

Choose a reason for hiding this comment

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

좀 더 구분했다고 해야될까요?
저는 주로 p태그를 사용하는 이유가 글로만 이루어진 문단이라면 사용하고 있습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] 시맨틱 태그 + MetaData 수정
3 participants