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

사용되지 않는 API 제거 완료 #442

Merged
merged 15 commits into from
Aug 3, 2024
Merged

사용되지 않는 API 제거 완료 #442

merged 15 commits into from
Aug 3, 2024

Conversation

limehee
Copy link
Collaborator

@limehee limehee commented Aug 3, 2024

Summary

#441

Members 페이지에서 사용되지 않는 API를 제거하여 코드베이스를 정리하고 유지보수성을 향상시키기 위한 작업입니다. 사용 예정이나 미연결된 도메인을 제외한 모든 불필요한 API를 제거합니다.

Tasks

도메인 API
동아리 지원 지원서 삭제
동아리 지원 삭제된 지원서 조회하기
모집 공고 모집 공고 삭제
모집 공고 삭제된 모집 공고 조회하기
도서 삭제된 도서 조회하기
로그인 멤버 토큰 삭제
알림 알림 생성
알림 알림 삭제
알림 삭제된 알림 조회하기
직책 삭제된 직책 조회하기
일정 삭제된 일정 조회하기
파일 업로드 뉴스 사진 업로드
파일 업로드 양식 업로드
미연결 도메인 뉴스
미연결 도메인 로그인 시도 로그
미연결 도메인 채용 공고
미연결 도메인 이메일

@limehee limehee linked an issue Aug 3, 2024 that may be closed by this pull request
1 task
@limehee limehee self-assigned this Aug 3, 2024
@limehee limehee requested review from SongJaeHoonn and mingmingmon and removed request for SongJaeHoonn August 3, 2024 02:59
@limehee limehee added the 🔨 Refactor 코드 수정 및 개선 label Aug 3, 2024
@limehee limehee requested a review from SongJaeHoonn August 3, 2024 03:00
@SongJaeHoonn
Copy link
Contributor

지원서 삭제와, 모집 공고 삭제같은 api가 사라진다면, 실제로 삭제해야 될 상황에는 직접 DB에서 조작하는 것인가요?

@limehee
Copy link
Collaborator Author

limehee commented Aug 3, 2024

지원서 삭제와, 모집 공고 삭제같은 api가 사라진다면, 실제로 삭제해야 될 상황에는 직접 DB에서 조작하는 것인가요?

이번 작업에서 제외된 API 및 도메인은 프론트를 담당하고 있는 @Jeong-Ag 님과 협의된 사항임을 알립니다.

지원서 삭제 API

동아리 내외부 인원으로부터 지원서를 받아 검토 후, 합격 처리를 하는 과정을 거치고 있습니다. 공고별로 지원받았던 서류를 보존할 필요가 있어 지원서 삭제 API를 제거했습니다.

모집 공고 삭제 API

모집 공고는 동아리 회원 및 운영진과 면밀한 검토 후에 게시되고 있습니다. 모집 공고가 실수로 게시되는 일은 없다고 판단되며, 세부 사항에 대한 수정이 필요할 경우 별도의 수정 API를 통해 변경할 수 있어 문제가 없다고 여겨집니다.

@mingmingmon
Copy link
Collaborator

파일 업로드 API 중 양식 업로드는 활동에서 과제 양식을 그룹장이 업로드할 때 사용될 API라고 알고 있었습니다.
현재 활동 부분 API 작업을 새로 진행할 예정인 것으로 알고 있는데
양식 업로드 API는 활동 API 작업을 새로 할 때 알맞은 형태로 추가될 예정인건가요?

@limehee
Copy link
Collaborator Author

limehee commented Aug 3, 2024

파일 업로드 API 중 양식 업로드는 활동에서 과제 양식을 그룹장이 업로드할 때 사용될 API라고 알고 있었습니다. 현재 활동 부분 API 작업을 새로 진행할 예정인 것으로 알고 있는데 양식 업로드 API는 활동 API 작업을 새로 할 때 알맞은 형태로 추가될 예정인건가요?

양식 업로드 API는 프로젝트 초기 설계 당시 동아리 전체에서 관리 목적으로 사용되는 양식 파일을 업로드하는 용도로 만들어졌습니다. 하지만, 현재 동아리에서 공통적으로 사용 중인 양식이 없기에 삭제를 결정하게 되었습니다. 활동 도메인에서 파일 업로드가 필요하게 된다면 API를 추가로 작성하는게 좋아보입니다. (기존 양식 업로드 API는 저장 위치도 /forms로 되어있기에 활동 도메인에 적절하지 않습니다)

@limehee limehee merged commit eaf0d5c into develop Aug 3, 2024
3 checks passed
@limehee limehee deleted the refactor/#441 branch August 3, 2024 09:39
@limehee limehee mentioned this pull request Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 수정 및 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

사용되지 않는 API 제거
3 participants