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

✨ [Merge] Agenda(행사)기능 추가 #1013

Merged
merged 113 commits into from
Sep 10, 2024
Merged

✨ [Merge] Agenda(행사)기능 추가 #1013

merged 113 commits into from
Sep 10, 2024

Conversation

AreSain
Copy link
Contributor

@AreSain AreSain commented Sep 10, 2024

📌 개요

  • 평가가 부족한 42를 개선하고자 평가 포인트를 사용해서 티켓을 발급받고 그 티켓으로 행사에 참여하는 기능을 작성했습니다.
  • 참여도 증가를 위해 티켓을 사용하지 않아도 행사를 개최할 수 있으며, 티켓을 사용해서 개최시 42측의 지원을 받아 칭호 및 업적을 부여할 수 있습니다.

💻 작업사항

카테고리 엔드포인트 메서드 URL
대회 대회 상세 조회 GET /agenda
대회 생성하기 (개인, 비공식) POST /agenda/request
진행중 대회 전체 조회 GET /agenda/confirm
모집중 대회 전체 조회 GET /agenda/open
지난 대회 보기 GET /agenda/history
대회 확정하기 PATCH /agenda/confirm
대회 종료 및 시상하기 PATCH /agenda/finish
대회 취소하기 PATCH /agenda/cancel
대회 공지사항 대회 공지사항 만들기 POST /agenda/announcement
대회 공지사항 전체 조회 GET /agenda/announcement/list
개인 페이지 로그인 사용자 프로필 정보 조회 GET /agenda/profile/info
내 프로필 상세 조회 GET /agenda/profile
프로필 상세 조회 GET /agenda/profile/{intraId}
42Intra 프로필 조회 GET /agenda/profile/intra/{intraId}
내 프로필 변경 PATCH /agenda/profile
참여 중인 대회 보기 GET /agenda/profile/current/list
참여 했던 대회 보기 GET /agenda/profile/history/list/{intraId}
개최했던 대회 조회 GET /agenda/host/history/list/{intraId}
개최중인 대회 조회 GET /agenda/host/current/list/{intraId}
내 팀 간단 조회 GET /agenda/team/my
팀 생성하기 POST /agenda/team
모집중인 팀 리스트 조회 GET /agenda/team/open/list
확정된 팀 리스트 조회 GET /agenda/team/confirm/list
팀 상세 조회 GET /agenda/team
팀 참가하기 POST /agenda/team/join
팀장이 취소(폭파)하기 PATCH /agenda/team/cancel
팀원이 나가기 PATCH /agenda/team/drop
팀 확정하기 PATCH /agenda/team/confirm
팀 내용 수정하기 PATCH /agenda/team
티켓 티켓 발급 시작하기 POST /agenda/ticket
티켓 발급 완료하기 PATCH /agenda/ticket
가진 티켓 확인하기 GET /agenda/ticket
티켓 히스토리 확인하기 GET /agenda/ticket/history
어드민 대회 대회 전체 조회 GET /agenda/admin/request/list
대회 목록 조회 GET /agenda/admin/list
대회 수정/삭제 PATCH /agenda/admin/request
어드민 개인 페이지 개인 페이지 수정하기 PATCH /agenda/admin/profile
어드민 공지사항 공지사항 전체 조회 GET /agenda/admin/announcement/list
공지사항 수정/삭제 PATCH /agenda/admin/announcement
어드민 팀 팀 전체 목록 조회 GET /agenda/admin/team/list
팀 상세 조회 GET /agenda/admin/team
팀 수정 PATCH /agenda/admin/team
팀 취소 PATCH /agenda/adimin/team/cancel
어드민 티켓 모든 티켓 조회 GET /agenda/admin/ticket/list
티켓 생성 POST /agenda/admin/ticket
티켓 수정 PATCH /agenda/admin/ticket
이벤트 대상
AGENDA 취소 해당 AGENDA 참가자
AGENDA 종료 해당 AGENDA 참가자
AGENDA 확정 해당 AGENDA 참가자
팀 CONFIRM 해당 팀 참가자
팀 폭파 해당 팀 참가자
팀에 누군가 참가 해당 팀 참가자
팀에 누군가 나감 해당 팀 참가자
AGENDA 최대 팀 최대 도달 AGENDA 개최자
AGENDA 최대 팀 최소 도달 AGENDA 개최자

✅ 변경로직

  • 기존 SLACK과 EMAIL을 공통 모듈로 꺼내고 리팩토링 했습니다. (다른 모듈에서 공통으로 사용하게 하기 위함입니다.)
  • ci 에 TestFixture를 추가했습니다. (cd에는 있었지만 ci에 없어서 배포 도중 문제가 생기는 경우가 발생했었습니다.)
  • README.md 수정 (AwsArchitecture, 모듈 구조도, 7기 워크플로우, 5기까지ERD, 6기ERD, 7기ERD를 추가했습니다.)

sujin and others added 30 commits February 18, 2024 23:04
kimjieun0301 and others added 16 commits August 30, 2024 14:03
@AreSain AreSain added documentation Improvements or additions to documentation enhancement New feature or request test labels Sep 10, 2024
@AreSain AreSain added this to the 7th Sprint 5 milestone Sep 10, 2024
Copy link

codecov bot commented Sep 10, 2024

Codecov Report

Attention: Patch coverage is 60.52632% with 270 lines in your changes missing coverage. Please review.

Project coverage is 74.21%. Comparing base (6e38756) to head (f1ee767).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...g-auth/src/main/java/gg/auth/FortyTwoAuthUtil.java 0.00% 43 Missing ⚠️
...in/java/gg/utils/file/handler/AwsImageHandler.java 0.00% 38 Missing ⚠️
...main/java/gg/utils/sns/slack/SlackbotApiUtils.java 22.50% 30 Missing and 1 partial ⚠️
...va/gg/utils/sns/slack/AsyncSlackMessageSender.java 5.26% 18 Missing ⚠️
.../src/main/java/gg/auth/utils/RefreshTokenUtil.java 0.00% 15 Missing ⚠️
gg-data/src/main/java/gg/data/agenda/Ticket.java 77.41% 12 Missing and 2 partials ⚠️
...utils/src/main/java/gg/utils/external/ApiUtil.java 0.00% 14 Missing ⚠️
gg-data/src/main/java/gg/data/agenda/Agenda.java 91.27% 2 Missing and 11 partials ⚠️
...-data/src/main/java/gg/data/agenda/AgendaTeam.java 85.71% 5 Missing and 6 partials ⚠️
...pi/user/noti/service/sns/SlackPartybotService.java 0.00% 10 Missing ⚠️
... and 16 more
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1013      +/-   ##
============================================
- Coverage     75.82%   74.21%   -1.62%     
- Complexity     1090     1221     +131     
============================================
  Files           365      393      +28     
  Lines          4811     5468     +657     
  Branches        391      439      +48     
============================================
+ Hits           3648     4058     +410     
- Misses          991     1220     +229     
- Partials        172      190      +18     
Flag Coverage Δ
integrationTest 69.18% <59.64%> (-0.83%) ⬇️
unitTest 35.71% <30.40%> (-1.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@Kimhan-nah Kimhan-nah 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
Contributor

@wken5577 wken5577 left a comment

Choose a reason for hiding this comment

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

고생하셨어요!

@AreSain AreSain merged commit 4dd0925 into main Sep 10, 2024
5 of 6 checks passed
@ghyen ghyen self-requested a review September 10, 2024 13:39
@ghyen
Copy link
Contributor

ghyen commented Sep 10, 2024

수고 많으셨습니다~!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request test
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants