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

apply sem001 #1226

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

apply sem001 #1226

wants to merge 3 commits into from

Conversation

wjeongchoi
Copy link
Contributor

요약 *

It closes #1225

학기 정보 가져오는 현재 학기 정보를 보여줘야 하는 화면에 SEM001 적용

고민되는 부분

어떤 api를 호출할 때 param으로 semesterId를 넣어야 하는 경우가 간혹 있는데(예: MemberManageFrame에서 CLB010 사용), 한번에 호출해서 처리하려고 하면 문제가 생기는 것을 발견함 (같은 파일에서 clubId 다른 api 통해서 가져다 쓰는 건 잘 됨) 이를 어떻게 해결할지
생각해본 방법

  • 백엔드에서 semesterId 비우면 현재 학기로 처리하도록 설정하기
  • 전역변수로 semester 정보 관리
  • frame depth 하나 더 들어가서 쓰도록 수정

현재 학기 정보를 사용하는 화면이 생각보다는 적었는데, provider 이용해서 전역변수로 관리해야할지도 고민

스크린샷

이후 Task *

  • SEM001 적용 과정에서 ClubRegistrationButton과 RegisterInfo 코드가 매우 유사한 것을 발견함. 별도 이슈로 분리해서 수정해보면 좋을 것 같음.

@wjeongchoi wjeongchoi added enhancement New feature or request front-end labels Nov 22, 2024
@wjeongchoi wjeongchoi self-assigned this Nov 22, 2024
@wjeongchoi wjeongchoi linked an issue Nov 22, 2024 that may be closed by this pull request
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request front-end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply SEM001
1 participant