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

chore: update i18next typed string #62

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

minsoo-web
Copy link

@minsoo-web minsoo-web commented Jun 21, 2023

안녕하세요, 우연히 깃허브가 추천해준 레포로 떠서 둘러보던 중에 i18next에 typescript 를 적용시킬 수 있는 방법이 떠올라서 기여하면 좋겠다는 생각에 PR을 남기게 되었습니다!!

Overview

I Add d.ts file for using i18next by typed.

Screenshots

Before

image

After

image

Questions

next-i18next.config.js 파일을 import 해서 next.config.js 의 i18n 설정을 넣어주면 다른 설정을 굳이 할 필요가 없는데,
지금처럼 Redirect 처리를 하시거나, locale을 확인하는 등의 코드들로 작업하신 이유가 있으신가요?!

const { i18n } = require("./next-i18next.config");
const webpack = require("webpack");

const nextConfig = {
  i18n,
  reactStrictMode: true,
  // ...
}

@dev-redo
Copy link
Collaborator

안녕하세요! ask resume 서비스를 발전시키는 데 기여해주셔서 감사합니다 :)
늦어도 이번주 주말 안으로 해당 pr을 확인하고, 질문 주신 부분에 대해서도 답을 드릴 수 있게끔 하겠습니다.
다시 한 번 감사의 말씀을 올립니다 :)

@minsoo-web
Copy link
Author

안녕하세요! ask resume 서비스를 발전시키는 데 기여해주셔서 감사합니다 :) 늦어도 이번주 주말 안으로 해당 pr을 확인하고, 질문 주신 부분에 대해서도 답을 드릴 수 있게끔 하겠습니다. 다시 한 번 감사의 말씀을 올립니다 :)

안녕하세요! 부담갖지 마시구 관리하시기 편하신 시간대에 봐주시면 감사하겠습니다!!

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