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

[FE] fix: 신입모집 기수의 숫자를 업데이트한다. #13

Open
4 tasks
2yunseong opened this issue Feb 13, 2024 · 7 comments
Open
4 tasks

[FE] fix: 신입모집 기수의 숫자를 업데이트한다. #13

2yunseong opened this issue Feb 13, 2024 · 7 comments
Assignees

Comments

@2yunseong
Copy link
Collaborator

2yunseong commented Feb 13, 2024

  • 신입모집 기수의 숫자를 26기에서 27기로 업데이트 해야합니다.
  • 업데이트 하며 상수화가 되어있지 않은 파일은 상수화 하거나 다른 대책을 고안한다.

To do

  • react-app-env에 타입설정
  • 빌드 시 .env에 포함 + 문서화
  • 기존에 constants 파일에 있는 상수 제거 후 환경변수 화
  • email로 보내는 html 문서에 있는 기수 상수화
@loopy-lim
Copy link
Collaborator

이 상수데이터를 import말고 먼가.... compiler단에서 넣는 방식은 없을까요?

@2yunseong
Copy link
Collaborator Author

2yunseong commented Feb 13, 2024

환경변수화? 하고

  • react-app-env에 타입설정
  • 빌드 시 .env에 포함 + 문서화

@loopy-lim
Copy link
Collaborator

오 좋은거 같은데 진심으로 다른 분들은 어떻게 생각하신가요?
@bada308 @geongyu09 @baegyeong

@2yunseong
Copy link
Collaborator Author

궁금한게 email.html에는 어떻게 동적으로 주입해주죠?
js하나 만들어줘야하나

@loopy-lim
Copy link
Collaborator

email.html에서는 BE에서 주입을 해주어서, 저희가 {{generation}} 이런식으로 정해져있는 text를 주어도 좋아요!

@geongyu09
Copy link
Collaborator

환경변수로 관리하면 훨씬 좋을 것 같아요! 더러운(?) import 문 사용 없이도 관리가 수월할 것 같네요

@baegyeong
Copy link
Collaborator

오오 그런 상수 데이터를 환경변수화 하는 것은 생각해본 적이 없었는데 관리하는 데 있어서 좋은 것 같아요!

@loopy-lim loopy-lim self-assigned this Feb 21, 2024
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

No branches or pull requests

4 participants