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-22 hw complete #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

5-22 hw complete #9

wants to merge 1 commit into from

Conversation

YeomJongSeop
Copy link
Collaborator

@YeomJongSeop YeomJongSeop commented May 26, 2024

기본 과제
로그인 시에 이번에는 이메일로 로그인을 진행하였습니다

요구사항

  1. 아이디로 로그인 하도록 변경해주세요!
    1-1.아이디는 중복되지 않도록 중복 체크도 해주셔야 합니다!
  2. 추가적으로 회원가입시에 아래의 필드들을 추가적으로 받아 저장해주세요.
    3-1. email
    3-2. 멋사 기수를 나타내는 generation 값
    3-3. 남자 여자를 나타내는 gender 값

image
image

Login

{
"ID": “yjs6244”,
"password": "yjs1234"
}
image

@YeomJongSeop YeomJongSeop requested a review from kseysh May 26, 2024 14:23
Copy link
Member

@kseysh kseysh left a comment

Choose a reason for hiding this comment

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

앗 이거 까먹고 못했네요 고생하셨습니다!!

user = self.create_user(
email,
ID=ID,
Copy link
Member

Choose a reason for hiding this comment

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

장고에서는 최대한 스네이크 케이스를 활용해주세요!

@kseysh kseysh added the 통과 label May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants