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.

Sign up for GitHub

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

Jump to bottom

[개선] 접수증 양식 변경 #155

Merged
merged 3 commits into from
Oct 13, 2024
Merged

[개선] 접수증 양식 변경 #155

merged 3 commits into from
Oct 13, 2024

Conversation

jyj1289
Copy link
Member

@jyj1289 jyj1289 commented Oct 12, 2024

  • 접수증에 대상구분을 추가하고 전형구분을 메인카테고리를 하도록 변경했어요.

🎫 관련 이슈

close #152


📄 개요

접수증 양식을 변경했습니다.


🔨 작업 내용

  • 접수증에 대상구분 값을 추가했습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

@jyj1289
perf(#152): 접수증 양식 변경
a982680 
- 접수증에 대상구분을 추가하고 전형구분을 메인카테고리를 하도록 변경했어요.
@jyj1289 jyj1289 requested a review from a team as a code owner October 12, 2024 06:48
@jyj1289 jyj1289 linked an issue Oct 12, 2024 that may be closed by this pull request
[개선] 접수증 양식 변경 #152
Closed
@github-actions GitHub Actions
Copy link

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 왜 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

@jyj1289 jyj1289 self-assigned this Oct 12, 2024
@cabbage16
Copy link
Member

cabbage16 commented Oct 12, 2024 •
edited
Loading

P4) 우연히 발견한건데 혹시 메서드에 오타 수정 가능한가요?
image

cabbage16
cabbage16 requested changes Oct 12, 2024
View reviewed changes
Copy link
Member

@cabbage16 cabbage16 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

P2)
다른 전형은 다 되는데
image

국가보훈대상자 중 교육지원대상자녀는 이름이 너무 길어서 다른칸이 이상하게 나오네요
image

@jyj1289
fix(#152): 오타수정
a266bda 
- FormController 에 generateProfOfApplication 메서드 오타를 generateProofOfApplication 메서드로 저장했어요.
@jyj1289
fix(#152): 국가보훈대상자 중 교육지원대상자녀 전형시 값 변경
c2df8f6 
- 국가보훈대상자 중 교육지원대상자녀 전형일 때 칸이 깨지는 것을 방지하기 위해 '국가보훈대상자'로 값을 바꾸었어요.
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Oct 12, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@cabbage16 cabbage16 merged commit f7d4f99 into develop Oct 13, 2024
2 checks passed
@cabbage16 cabbage16 deleted the perf/#152 branch October 13, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cabbage16 cabbage16 cabbage16 approved these changes

Assignees

@jyj1289 jyj1289

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[개선] 접수증 양식 변경
2 participants
@jyj1289 @cabbage16