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] 운동 시작, 실시간 좋아요 #20

Merged
merged 10 commits into from
Aug 31, 2023
Merged

[api] 운동 시작, 실시간 좋아요 #20

merged 10 commits into from
Aug 31, 2023

Conversation

bngsh
Copy link
Collaborator

@bngsh bngsh commented Aug 17, 2023

😎 작업 내용

  • 운동 시작 API 수정 (서버 단에서 히스토리 생성)
  • 내가 팔로우하는 유저 중 운동 중인 사람에게 좋아요 보내는 API 추가

🧐 변경된 내용

  • Walkie 객체의 status를 R(운동 중), N(온라인), F(오프라인), B(차단) 중의 값으로 통일

🥳 동작 화면

  • 동작 화면 없음

🤯 이슈 번호

  • 이슈 없음

@bngsh bngsh added 병희 api api 작성 관련 작업 labels Aug 17, 2023
@bngsh bngsh self-assigned this Aug 17, 2023
@github-actions
Copy link

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

Copy link
Member

@peter-j0y peter-j0y left a comment

Choose a reason for hiding this comment

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

LGTM!

Comment on lines 30 to 50
@NotNull
@Column(name = "end_time", nullable = false)
@Column(name = "end_time")
private Date endTime;

@NotNull
@Column(name = "total_time", nullable = false)
@Column(name = "total_time")
private Integer totalTime;

@NotNull
@Column(nullable = false)
private Integer calorie;

@NotNull
@Column(nullable = false)
private Integer step;

@NotNull
@Column(nullable = false)
Copy link
Member

Choose a reason for hiding this comment

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

이거 다 nullable하게 만드신 이유가 있으신가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

러닝 막 시작하는 요청에 시작 시간만 있는 히스토리를 만들어줘야해서 널러블로 바꿨습니다!!

@bngsh bngsh merged commit 0bef286 into main Aug 31, 2023
@bngsh bngsh linked an issue Aug 31, 2023 that may be closed by this pull request
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api api 작성 관련 작업 병희
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[API] 러닝 화면 관련 API 구현
2 participants