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

[Feat] 목표 체계 제거 #252

Merged
merged 2 commits into from
May 18, 2024
Merged

[Feat] 목표 체계 제거 #252

merged 2 commits into from
May 18, 2024

Conversation

funnysunny08
Copy link
Contributor

🚩 관련 이슈

📋 구현 기능 명세

  • Goal 관련 로직 제거

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지

  • 어떤 부분에 리뷰어가 집중해야 하는지

  • 개발하면서 어떤 점이 궁금했는지

📸 결과물 스크린샷

@Pattern(regexp = "^[\\S][가-힣a-zA-Z0-9\\s]{0,20}$", message = "위니 피드 제목 형식에 맞지 않습니다.")

🛠️ 테스트

  • 테스트

🚀 API Endpoint

  • /

@funnysunny08 funnysunny08 self-assigned this May 18, 2024
sss4920
sss4920 approved these changes May 18, 2024
View reviewed changes
Copy link
Contributor

@sss4920 sss4920 left a comment

Choose a reason for hiding this comment

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

ㅠㅠ 굿굿 만약 빵꾸난곳 있으면 제가 로직 올리면서 확인 한번 더 해보고 올리겠습니당

@funnysunny08 funnysunny08 merged commit a427211 into dev May 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sss4920 sss4920 sss4920 approved these changes

@alpha-kwhn alpha-kwhn Awaiting requested review from alpha-kwhn

Assignees

@funnysunny08 funnysunny08

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] 목표 체계 제거
2 participants
@funnysunny08 @sss4920