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

[#85] 문제 보기 #86

Merged
merged 2 commits into from
Nov 21, 2023
Merged

[#85] 문제 보기 #86

merged 2 commits into from
Nov 21, 2023

Conversation

dmdmdkdkr
Copy link
Collaborator

한 일

  • ContestBreadCrumb에서 key props를 추가하는 로직을 추가했습니다.
  • 문제 보기 페이지인 ProblemPage를 작성하였습니다.
    • TODO: api를 적용하는 태스크를 진행하며 수정할 계획입니다.
      image

@dmdmdkdkr dmdmdkdkr added the FE fe 개발 label Nov 21, 2023
@dmdmdkdkr dmdmdkdkr self-assigned this Nov 21, 2023
Copy link
Collaborator

@mahwin mahwin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다 😎

Copy link
Collaborator

@dev2820 dev2820 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

사소하지만 ProblemPage에 currentProblemId는 아직 useState를 사용하지 않아도 되는 것으로 보이네요.

아마 미래에 변경할 것을 고려해 적용해두었을 것 같은데, 저는 당장 사용하지 않을 코드는 제거하는 편이긴 해요.

그것이 논리 흐름을 파악하기 어렵게 만들거든요. 또 막상 때가 되면 사용하지 않고 폐기하기도 하고, 나중에 보니 더 좋은 이름이 떠오르기도 합니다.

하지만 크게 중요하지 않은 부분이라 여겨 approve 합니다. 고생하셨어요

@dmdmdkdkr dmdmdkdkr merged commit a84e704 into fe-dev Nov 21, 2023
@dmdmdkdkr dmdmdkdkr deleted the 85-문제-보기 branch November 21, 2023 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE fe 개발
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants