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

[개선] 금연서약서 형식변경 및 전체 PDF 서식 변경 #122

Merged
merged 5 commits into from
Sep 7, 2024
Merged

[개선] 금연서약서 형식변경 및 전체 PDF 서식 변경 #122

merged 5 commits into from
Sep 7, 2024

Conversation

jyj1289
Copy link
Member

@jyj1289 jyj1289 commented Sep 7, 2024

🎫 관련 이슈

close #120


📄 개요

금연서약서 및 전체 PDF를 변경했습니다.


🔨 작업 내용

  • 서약서 템플릿을 만들었어요.
  • 모든 템플릿에 서식 번호를 붙였어요.

🏁 확인 사항

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

🙋🏻 덧붙일 말

MinjuN07 and others added 5 commits September 7, 2024 11:29
@MinjuN07
feat(#120): 입학 원서 서류 PDF 서식 명시
95cfe1a 
- 입학 원서 서류 중 존재하지 않는 서식을 추가했어요.
@MinjuN07
feat(#120): 입학 전형 서류 서약서 퍼블리싱
fb0a470 
- 입학 전형 서류의 변경된 서식에 따라 금연 서약서를 다시 퍼블리싱 하였습니다.
@jyj1289
perf(#120): 서약서 템플릿 추가
62453ce 
- 금연 동의서가 아닌 서약서로 변경되어서 서약서 템플릿을 추가하고 적용했어요.
@jyj1289
perf(#120): 서약서 템플릿 추가
48bbaeb 
- 특별전형도 금연 동의서가 아닌 서약서로 변경되어서 서약서 템플릿을 추가하고 적용했어요.
@jyj1289
refactor(#120): 하단 텍스트 중앙정렬
c8c6b1b 
- 2025학년도 입학전형 요강 학교장 추천서에 따라 "부산소프트웨어마이스터고등학교장 귀하" 텍스트를 중앙정렬했어요.
@jyj1289 jyj1289 requested a review from a team as a code owner September 7, 2024 09:17
@jyj1289 jyj1289 linked an issue Sep 7, 2024 that may be closed by this pull request
[개선] 금연서약서 형식변경 및 전체 PDF 서식 변경 #120
Closed
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 7, 2024

이렇게 리뷰해주세요.

🖐️ 규칙

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

🥬 우선순위

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

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

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

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Sep 7, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
4.6% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@jyj1289 jyj1289 self-assigned this Sep 7, 2024
@jyj1289 jyj1289 merged commit 44b8d43 into develop Sep 7, 2024
2 of 3 checks passed
@jyj1289 jyj1289 deleted the perf/#120 branch September 7, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jyj1289 jyj1289

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[개선] 금연서약서 형식변경 및 전체 PDF 서식 변경
2 participants
@jyj1289 @MinjuN07