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

Merge: 토익 문제 수정 및 삭제 적용 #43

Merged
merged 8 commits into from
Feb 25, 2024
Merged

Merge: 토익 문제 수정 및 삭제 적용 #43

merged 8 commits into from
Feb 25, 2024

Conversation

Zamoca42
Copy link
Member

@Zamoca42 Zamoca42 commented Feb 25, 2024 •
edited
Loading

PR 체크리스트

아래 항목을 확인해 주세요:

  • 커밋 메시지가 우리의 가이드라인을 따르고 있는지 확인하세요
  • 변경 사항에 대한 테스트가 추가되었는지 확인하세요 (버그 수정 / 기능 추가)
  • 문서가 추가되거나 업데이트되었는지 확인하세요 (버그 수정 / 기능 추가)

PR 유형

이 PR은 어떤 종류의 변경을 가져오나요?

  • 버그 수정
  • 새로운 기능 추가
  • 코드 스타일 업데이트 (서식, 로컬 변수)
  • 리팩터링 (기능 변경 없음, API 변경 없음)
  • 빌드 관련 변경
  • CI 관련 변경
  • 문서 내용 변경
  • 애플리케이션 / 인프라 변경
  • 기타... 설명:

관련 이슈

이슈 번호: #35

현재 동작은 무엇인가요?

  • 토익 문제를 생성, 조회

새로운 동작은 무엇인가요?

  • 토익 문제를 생성, 조회, 수정, 삭제

이 PR은 호환성 변경을 도입하나요?

  • 예
  • 아니요

기타사항

예시: 토익 id 7의 문제 8의 답을 B에서 A로 수정

스크린샷 2024-02-25 오후 4 24 04

스크린샷 2024-02-25 오후 4 23 45

수정 성공

스크린샷 2024-02-25 오후 4 24 28

예시: 토익 id 7의 문제를 삭제

삭제요청

스크린샷 2024-02-25 오후 4 50 18

삭제 성공시

스크린샷 2024-02-25 오후 4 50 38

@Zamoca42
Feat: 토익 문제에서 ResponseEntity 추가
b10e3c0 
- CRUD 기능을 위한 API 추가
- 토익 문제를 추가하고 수정하고 삭제하는 기능을 추가했습니다.
- Response Entity를 적용하여 클라이언트에게 응답을 보내는 기능을 추가했습니다.

Related to #35
@Zamoca42
Refact: 업데이트 이름 변경
d428d31 
- Data로 끝나는 Dto 이름을 변경
- 문제 수정에 수정 요청 바디 추가
Related to #35
@Zamoca42
Feat: 토익 문제 수정 인터페이스 추가
a6f5c50 
- 수정 요청용 Body dto 추가
- 토익 문제 수정 시 필요한 인터페이스 추가
Related to #35
@Zamoca42 Zamoca42 changed the title Feat/#35 Merge: 토익 문제 수정 및 삭제 적용 Feb 25, 2024
future9061
future9061 approved these changes Feb 25, 2024
View reviewed changes
Copy link
Member

@future9061 future9061 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

수고하셨습니다!

@Zamoca42
Refact: 토익 문제 리팩터링
33cb986 
- 토익 문제 수정, 삭제시 매니저 권한이 필요하도록 수정

Related to #35
@Zamoca42
Feat: 문제 상세 조회 추가
a933321 
- 토익 내의 문제 상세 조회 기능을 추가했습니다.
- 문제 상세 조회를 위한 API를 추가했습니다.
- 메서드 이름을 변경했습니다.

Related to #35
@Zamoca42
Refact: 토익 문제 조회 API 수정
86fae1e 
- 토익 문제 조회 API에서 Query 파라미터 옵션 추가
- 기본으로 공개된 문제만 조회하도록 변경

Related to #35
@Zamoca42 Zamoca42 merged commit 36b305d into main Feb 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@future9061 future9061 future9061 approved these changes

@sheepdog13 sheepdog13 Awaiting requested review from sheepdog13

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Zamoca42 @future9061