Front/master to master - [merged]

[seoyoung059]

In GitLab by @wnso-kim on Aug 14, 2024, 14:56

Merges front/master -> master

:link: 해당 MR과 연결된 JIRA의 태스크는 무엇인가요?

#

🔍️ 해당 MR을 통해 해결하려는 문제가 무엇인가요?

어떤 기능을 구현한 건지, 이슈 대응이라면 어떤 이슈인지 MR이 열리게 된 계기와 목적을 Reviewer들이 쉽게 이해할 수 있도록 적어 주세요 다이어그램, 피그마를 첨부해도 좋아요

• MR 템플릿 자동 설정 테이블 제거와 템플릿 보완

✨ 해당 MR에서 핵심적으로 변경된 사항은 무엇일까요?

문제를 해결하면서 주요하게 변경된 사항들을 적어 주세요

• 자동 설정 값 제거
• 템플릿 가독성 증가

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?

없으면 "없음" 이라고 기재해 주세요

• 없음

📝 작업중 고찰하신게 있나요?

이번에 특정 기술 도입 또는 방향성 선택의 이유가 있다면, 그 부분에 대해서 설명해주세요!

DevOcean Blog를 참고하여 템플릿의 가독성을 높였습니다. 기존 Issue 템플릿에서 사용하던 자동 설정값이 PR 템플릿에는 적용이 되지 않는다는 사실을 알게 되었습니다. 그로인해, 설정 테이블을 제거했습니다.

🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요

개발 과정에서 다른 분들의 의견은 어떠한지 궁금했거나 크로스 체크가 필요하다고 느껴진 코드가 있다면 남겨주세요

없으면 "없음" 이라고 기재해 주세요

• 없음

✅ 제출 전 필수 확인 사항

• [ ] 빌드가 되는 코드인가요?
• [ ] 버그가 발생하지 않는지 충분히 테스트 해봤나요?

📌 MR 진행 시 이러한 점들을 참고해 주세요

• Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
• Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
• Comment 작성 시 Prefix로 P1, P2, P3 를 적어 주시면 Assignee가 보다 명확하게 Comment에 대해 대응할 수 있어요
  • P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
  • P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
  • P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

[seoyoung059]

In GitLab by @wnso-kim on Aug 14, 2024, 14:59

added 3 commits

• ed04f55d - :sparkles: Feat: 메인페이지 수정
• c52d6496 - :sparkles: Feat: 메인페이지 이미지 수정
• 0ba96b54 - Merge branch 'front/MainPageFix' into 'front/master'

Compare with previous version

[seoyoung059]

In GitLab by @wnso-kim on Aug 14, 2024, 15:08

mentioned in commit 28a9e386744fb99f495ac91e82780390098a1f9c