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

학생 폼 제출할 때 재 활용성을 올립니다. #396

Merged
merged 2 commits into from
Jul 9, 2024

Conversation

Huuuunee
Copy link
Member

@Huuuunee Huuuunee commented Jul 9, 2024

💡 배경 및 개요

지금 선생님이 만든 인증제 폼 리스트 조회 API가 존재하지 않아 클라이언트에서 하드코딩을 하던걸 param으로 수정합니다

Resolves: #395

📃 작업내용

PathVariable 으로 받던 uuid를 Param으로 받게끔 변경합니다

🙋‍♂️ 리뷰노트

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

@Huuuunee Huuuunee added 1️⃣ Priority: High 우선순위 - 상 ⚡️ Simple 간단한 변경사항 labels Jul 9, 2024
@Huuuunee Huuuunee self-assigned this Jul 9, 2024
@Huuuunee Huuuunee linked an issue Jul 9, 2024 that may be closed by this pull request
Copy link
Member

@chanwoo00106 chanwoo00106 left a comment

Choose a reason for hiding this comment

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

LGTM

@Huuuunee Huuuunee merged commit 538ca1d into develop Jul 9, 2024
1 check passed
@Huuuunee Huuuunee deleted the feature/395-submit-form-active branch July 9, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1️⃣ Priority: High 우선순위 - 상 ⚡️ Simple 간단한 변경사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

폼 제출을 할때 Active된 폼으로 매핑 되도록 합니다.
3 participants