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

[5기 - 최정은] SpringBoot Part3 Weekly Mission 제출합니다. #964

Open
wants to merge 16 commits into
base: JeongeunChoi/week3
Choose a base branch
from

Conversation

JeongeunChoi
Copy link

@JeongeunChoi JeongeunChoi commented Nov 4, 2023

📌 과제 설명

(기본) 바우처 서비스 관리페이지 개발하기

  • Spring MVC를 적용해서 thymeleaf 템플릿을 설정해보세요.
  • 커맨드로 지원했던 기능을 thymeleaf를 이용 해서 관리페이지를 만들고 다음 기능을 지원가능하게 해보세요
    • 조회페이지
    • 상세페이지
    • 입력페이지
    • 삭제기능

(기본) 바우처 서비스의 API 개발하기

  • Spring MVC를 적용해서 JSON과 XML을 지원하는 REST API를 개발해보세요

    • 전체 조회기능
    • 조건별 조회기능 (바우처 생성기간 및 특정 할인타입별)
    • 바우처 추가기능
    • 바우처 삭제기능
    • 바우처 아이디로 조회 기능
  • (보너스) 바우처 지갑용 관리페이지를 만들어보세요.**

👩‍💻 요구 사항과 구현 내용

  • 바우처 관련 기능들에 대해서 spring mvc와 thymeleaf 적용
  • 바우처 관련 기능들에 대해서 spring mvc와 json 적용
  • 지갑 관련 기능들에 대해서 spring mvc와 thymeleaf 적용
  • 바우처 종류로 바우처 찾기 기능 추가

✅ 피드백 반영사항

  • static final 컨벤션 Google Style Guide 참고
  • 시스템이 종료되어야하는 예외를 구분하여 처리
  • rest 설계 원칙 참고하여 RestVoucherController 수정
  • 각 컨트롤러 별 profile 설정

✅ PR 포인트 & 궁금한 점

  • 바우처 생성과 같은 API는 요청 후, 응답으로 받는 값을 현재 비워뒀는데, 이런 경우에는 보통 어떤 내용들이 들어가는지 궁금합니다!

- 조회페이지, 상세페이지, 입력페이지, 삭제기능
- 전체 조회기능, 바우처 타입별 조회 기능, 바우처 추가기능, 바우처 삭제기능, 바우처 아이디로 조회 기능
- CsvFileVoucherRepository와 MemoryVoucherRepository에 대해 구현하지 않아 추가
- 컨트롤러가 여러개 생겨남에 따라, 바우처 생성 요청 dto가 많아져 하나로 통일
@JeongeunChoi JeongeunChoi self-assigned this Nov 4, 2023
@JeongeunChoi JeongeunChoi changed the title [5기 - 최정은] SpringBoot Part2 Weekly Mission 제출합니다. [5기 - 최정은] SpringBoot Part1 Weekly Mission 제출합니다. Nov 4, 2023
@JeongeunChoi JeongeunChoi changed the title [5기 - 최정은] SpringBoot Part1 Weekly Mission 제출합니다. [5기 - 최정은] SpringBoot Part3 Weekly Mission 제출합니다. Nov 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant