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

Fix/profile #24

Merged
merged 43 commits into from
Nov 1, 2022
Merged

Fix/profile #24

merged 43 commits into from
Nov 1, 2022

Conversation

seuha516
Copy link
Contributor

  • request.user에서 user 정보를 사용할 수 있습니다

2

  • 프로필 보기

1
3

  • 프로필 수정 / 비밀번호 변경

  • 회원 탈퇴

  • App.tsx에 Header와 Footer 설정이 되어있습니다. 컴포넌트 위치들을 조정해 두었으니 새로 추가되는 컴포넌트도 적당한 위치에 넣어주세요

image

  • NotFound 컴포넌트

image

  • Loading 컴포넌트

  • Button과 input 컴포넌트를 따로 빼두었습니다

  • workoutlog에서 헤더 영역 삭제

@KJYoung
Copy link
Contributor

KJYoung commented Oct 31, 2022

  • user에 created field를 따로 추가하는 것보다 AbstractTimeStampedModel 을 상속하는게 낫지 않을까요?
    • 추가로 제가 알기론 아래와 같이 설정하면 보통 하는 작업을 수월하게 설정할 수 있습니다.
    • cf) https://tomining.tistory.com/145
    created = models.DateTimeField(auto_now_add=True)
    updated = models.DateTimeField(auto_now=True)
  • Button1,2,.. 등 번호로 명명한 것을 변경하거나, 만약 이들이 여러 곳에서 쓰일 것이라면 각 요소의 예시 사진과 함께 documentation을 남기면(귀찮겠지만...) 더 좋을 것 같긴 합니다!
  • 그외에는 다 좋은 것 같아요! 수고하셨습니다!

@seuha516
Copy link
Contributor Author

@KJYoung
수정했습니다. #29

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@KJYoung KJYoung left a comment

Choose a reason for hiding this comment

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

좋습니다.

@jihyungKo jihyungKo merged commit 983d357 into develop Nov 1, 2022
@seuha516 seuha516 deleted the fix/profile branch November 1, 2022 08:48
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.

3 participants