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

[새기능] 특례입학 원서 폼 및 접수증 만들기 #129

Merged
merged 18 commits into from
Sep 8, 2024

Conversation

jyj1289
Copy link
Member

@jyj1289 jyj1289 commented Sep 8, 2024

🎫 관련 이슈

close #119


📄 개요

특례입학 심사원서 및 접수증을 만들었습니다.


🔨 작업 내용

  • ExportFormUseCase에서 일반전형, 마이스터 인재전형, 특례입학전형을 구분하여 PDF를 추출하게 했습니다.
  • 최종제출을 한 상태에서 접수증 발급 API를 만들었습니다.
  • 확인서 및 특례입학 템플릿을 만들었습니다.

🏁 확인 사항

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

🙋🏻 덧붙일 말

jyj1289 and others added 17 commits September 6, 2024 13:40
- FormController에 접수증 생성 API를 작성했어요.
- proof-of-application.html을 상수로 추가했어요.
- GenerateProofOfApplicationUseCaseTest를 생성했어요.
- 접수증을 위해서 proof-of-application.html을 만들었어요.
- GenerateProofOfApplicationUseCase를 추가했어요.
- GenerateProofOfApplicationUseCaseSpringBootTest를 작성했어요.
- FormControllerTest에 접수증 발급 테스트 코드를 작성했어요.
- 입학 전형 서류에 따라 특례 입학 대상자 심사 원서를 퍼블리싱 하였습니다.
- 오타를 수정했어요.
- 접수증 발급 API를 작성했어요.
- 입학 전형 서류에 따라 특례 입학 대상자 확인서를 퍼블리싱 했어요.
- 특례입학 템플릿을 추가했어요.
- 입학 전형 서류에 따라 특례 입학 대상자 심사 원서의 누락된 부분을 미세 수정했어요.
- pdf 환경에서 :이 보이지 않는 것을 해결했어요.
- 특정 환경에서 글자들의 위치가 망가지는 것을 해결했어요.
- 입학전형 요강과 일치하게 스타일을 수정했어요.
@jyj1289 jyj1289 requested a review from a team as a code owner September 8, 2024 16:15
@jyj1289 jyj1289 linked an issue Sep 8, 2024 that may be closed by this pull request
6 tasks
Copy link

github-actions bot commented Sep 8, 2024

이렇게 리뷰해주세요.

🖐️ 규칙

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

🥬 우선순위

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

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

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

@jyj1289 jyj1289 self-assigned this Sep 8, 2024
@jyj1289 jyj1289 added the 개선 코드 리팩토링 label Sep 8, 2024
- 제출상태를 제외한 모든 원서에서 검증 조건을 최종제출, 승인, 접수가 아닌 원서를 검증하는 것으로 변경했어요.
Copy link

sonarcloud bot commented Sep 8, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@jyj1289 jyj1289 merged commit ee2ea32 into develop Sep 8, 2024
1 of 2 checks passed
@jyj1289 jyj1289 deleted the feat/#119 branch September 8, 2024 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
개선 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[새기능] 특례입학 원서 폼 및 접수증 만들기
2 participants