모든 URL 중앙화

[DaleSeo]

PR #196 에서 했던 것과 유사하게 다른 URL 들도 모두 data.js 파일로 추출한 후 필요한 컴포넌트 파일에서 임포트하도록 수정하였습니다.

Checklist before merging

• Link an issue with the pull request
• Ensure no errors or warnings on the browser console
• Avoid additional major pushes after approval (if necessary, request a new review)

[DaleSeo]

디스코드 URL이 여러 군데 있어도 이제는 동일하겠지 생각하고 있었는데 아니었네요. 😅 한 곳으로 빼기 잘 했다는 생각이 들었습니다. 코드 편집기의 모두 찾기 기능으로 일괄 바꿔치기 했더라면 이 놈은 놓칠뻔했습니다.

[DaleSeo]

본 PR의 목표와는 관계가 없지만 파일 이름이 컴포넌트 명과 일치하지 않아서 정정하였습니다.

[DaleSeo]

@DaleStudy/website 이렇게 쭉 URL을 뽑고 나니, <ds-language-section>와 <ds-participant-reviews-section>를 제외하고는 slot 때문에 index.html에 있던 HTML 마크업이 컴포넌트 내부로 들어가버렸습니다. 이 두 개의 컴포넌트에도 동일한 방식으로 리디자인 하는 것에 이견이 없으시면 후속 PR에서 작업하도록 하겠습니다.

[sounmind]

@DaleStudy/website 이렇게 쭉 URL을 뽑고 나니, <ds-language-section>와 <ds-participant-reviews-section>를 제외하고는 slot 때문에 index.html에 있던 HTML 마크업이 컴포넌트 내부로 들어가버렸습니다. 이 두 개의 컴포넌트에도 동일한 방식으로 리디자인 하는 것에 이견이 없으시면 후속 PR에서 작업하도록 하겠습니다.

이견 없습니다. 이쯤되니 Lit에서는 어떤 방식으로 코드를 작성하고 있을지 궁금하네요.

[sounmind]

LGTM! How do you find maintaining the code so far in this project?

[yolophg]

@DaleStudy/website 이렇게 쭉 URL을 뽑고 나니, <ds-language-section>와 <ds-participant-reviews-section>를 제외하고는 slot 때문에 index.html에 있던 HTML 마크업이 컴포넌트 내부로 들어가버렸습니다. 이 두 개의 컴포넌트에도 동일한 방식으로 리디자인 하는 것에 이견이 없으시면 후속 PR에서 작업하도록 하겠습니다.

동의합니다.