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

Open Graph 이미지, ALT 텍스트 추가 #18

Merged
merged 6 commits into from
Mar 22, 2024
Merged

Conversation

koremp
Copy link
Collaborator

@koremp koremp commented Mar 22, 2024

No description provided.

Copy link

vercel bot commented Mar 22, 2024

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

Name Status Preview Comments Updated (UTC)
call22nd ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 22, 2024 10:40am

@koremp koremp changed the title Open Graph 이미지 추가 Open Graph 이미지, ALT 텍스트 추가 Mar 22, 2024
@nestiank
Copy link
Collaborator

제22대 국회에게 요한다
제22대 국회에 요구한다

오타 수정 부탁드립니다

@koremp
Copy link
Collaborator Author

koremp commented Mar 22, 2024

제22대 국회에게 요한다 제22대 국회에 요구한다

오타 수정 부탁드립니다

어디가 오타인지 모르겠어요! 코드 상에서는 요구한다로 되어있어서요

@koremp koremp requested a review from nestiank March 22, 2024 09:09
@koremp koremp added the enhancement New feature or request label Mar 22, 2024
@koremp
Copy link
Collaborator Author

koremp commented Mar 22, 2024

제22대 국회에게 요한다 제22대 국회에 요구한다
오타 수정 부탁드립니다

어디가 오타인지 모르겠어요! 코드 상에서는 요구한다로 되어있어서요

아 메타태그였군요...

@koremp
Copy link
Collaborator Author

koremp commented Mar 22, 2024

제22대 국회에게 요한다 제22대 국회에 요구한다
오타 수정 부탁드립니다

어디가 오타인지 모르겠어요! 코드 상에서는 요구한다로 되어있어서요

아 메타태그였군요...

수정했습니다 @nestiank

@nestiank
Copy link
Collaborator

nestiank commented Mar 22, 2024

사소하지만 국회에게 -> 국회에 입니다.
현재 call22nd.works 들어가면 page title이 그렇게 되어 있어서 맞추면 좋을 것 같습니다 ㅠㅜ
수정해주시면 squash & merge 하겠습니다. 수고 많으세요

@koremp
Copy link
Collaborator Author

koremp commented Mar 22, 2024

사소하지만 국회에게 -> 국회에 입니다. 현재 call22nd.works 들어가면 page title이 그렇게 되어 있어서 맞추면 좋을 것 같습니다 ㅠㅜ 수정해주시면 squash & merge 하겠습니다. 수고 많으세요

title text를 상수로 빼서 한번만 수정하면 변경사항이 적용되도록 했습니다.

@nestiank
Copy link
Collaborator

현재 상태로는 테스트가 되지 않는데, 일단 merge하고 나중에 다시 수정을 하는 걸로 해야겠습니다.

@nestiank nestiank merged commit 2a9bc0c into develop Mar 22, 2024
4 checks passed
nestiank added a commit that referenced this pull request Mar 23, 2024
* 후보자가 답변하는 페이지 제작(call21st 코드 가져옴)

- 현재 token 유효성 검사 로직이 없음

* chore: add next-ga

* Open Graph 이미지, ALT 텍스트 추가 (#18)

* feat: add open graph image

* remove: remove opengraph tsx

* feat: add opengraph image and alt text

* feat: add og title and image to layout

* fix: fix og title typo

* refactor: add title text to meta and og

* 0322 버전에 맞게 텍스트 수정 및 컴포넌트 위치 재조정  (#20)

* refactor: 0322 버전에 맞게 메인 페이지 텍스트 수정 및 추가

* fix: fix typo and arrange logo position

* refactor: resize purple button

* refactor: add background to main ask component worker

* misc: padding 줄이기

* misc: 폰트가 한국어를 지원하도록 변경(`Noto Sans KR` 사용)

* misc: 컴포넌트 크기 및 위치 미세 조정

* fix: 로고가 투명하게 되는 문제 해결

---------

Co-authored-by: Juyoung Kim <[email protected]>

* fix: add openGraph to metaData

* 후보자들의 응답 결과를 보여주는 컴포넌트 제작(call21st 코드 가져옴) (#17)

* 후보자들의 응답 결과를 보여주는 컴포넌트 제작(call21st 코드 가져옴)

* chore: 불필요한 `console.log()` 제거

* misc: bullet 제거

* fix: 타입 오류 수정

* misc: 따옴표 표기 통일

* refactor: 불필요한 props 제거

* misc: 여백 조정

---------

Co-authored-by: Dokyun Lim <[email protected]>
@koremp koremp deleted the feat/create-og-img branch March 23, 2024 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants