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

feat: add OG Meta Tag #1

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

feat: add OG Meta Tag #1

wants to merge 9 commits into from

Conversation

scottXchoo
Copy link

@scottXchoo scottXchoo commented Jan 12, 2023

Summary

  • Faucet 페이지에 OG Meta Tag 추가함
  • 현재 이미지는 임의로 넣었고 디자이너가 만들어주시면 추가할 예정

Notes

  • Faucet 배포하는 Vercel이 Carina-Labs에 없어서 일단 환경변수로 빼지 못했음
  • 도메인 주소 환경변수로 빼야될 것 같음
  • 따로 라이브러리를 쓰지 않고 Head 안에 meta tag 추가하는 방식 채택함
  • 이미지 경로에서 시간을 조금 썼는데 https://도메인 주소/이미지 경로 이렇게 활용해야 됨

How to Test

image

@scottXchoo scottXchoo changed the title Feat/og tag feat: add OG Meta Tag Jan 12, 2023
Copy link

@helloheesu helloheesu left a comment

Choose a reason for hiding this comment

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

다시 살려주세요

frontend/public/index.html Outdated Show resolved Hide resolved
<meta
name="description"

Choose a reason for hiding this comment

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

description, og:description 둘 다 있어야합니다.
둘 중 하나 고르라면 오히려 description이 있어야합니다....
og 를 지원하는 많은 서비스들이 description을 fallback으로 삼는 경우가 많지만
구글 등 SEO는 og는 안 보기 때문입니다.

@@ -1,3 +0,0 @@
# https://www.robotstxt.org/robotstxt.html

Choose a reason for hiding this comment

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

robots.txt 를 일부러 지울 필요는...??
지운 의도가 무엇인가요?

Copy link
Author

Choose a reason for hiding this comment

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

불필요하다고 생각했는데 생각해 보니 일부러 지울 필요 없었던 것 같습니다 다시 살릴게요!

@scottXchoo scottXchoo marked this pull request as draft January 17, 2023 05:43
@helloheesu helloheesu marked this pull request as ready for review January 30, 2023 14:43
<meta property="og:title" content="Supernova Faucet" />
<meta property="og:description" content="Faucet is a tool for get testnet tokens in order to test Supernova" />
<meta property="og:site_name" content="Supernova" />
<meta property="og:image" content="http://faucet.dev-supernova.xyz/og.png" />

Choose a reason for hiding this comment

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

@boosik faucet 도 .ac 로 변경하나요?

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