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

πŸ”€ :: νšŒμ›κ°€μž…μ‹œμ— 학생과 μ„ μƒλ‹˜ κ΅¬λ³„ν•˜μ—¬ κΆŒν•œ λΆ€μ—¬ν•˜κΈ° #51

Merged
merged 6 commits into from
Apr 8, 2024
Merged

πŸ”€ :: νšŒμ›κ°€μž…μ‹œμ— 학생과 μ„ μƒλ‹˜ κ΅¬λ³„ν•˜μ—¬ κΆŒν•œ λΆ€μ—¬ν•˜κΈ° #51

merged 6 commits into from
Apr 8, 2024

Conversation

Umjiseung
Copy link
Member

πŸ’‘ κ°œμš”

νšŒμ›κ°€μž…μ„ ν• λ•Œ 학생과 μ„ μƒλ‹˜μ˜ κΆŒν•œμ„ κ΅¬λ³„ν•˜μ—¬ κΆŒν•œμ„ λΆ€μ—¬ν•˜μ˜€μŠ΅λ‹ˆλ‹€.

πŸ“ƒ μž‘μ—…λ‚΄μš©

  • User에 gauth_role 컬럼 μ‚­μ œ
  • MemberNotFound -> UserNotFound둜 λ³€κ²½
  • saveTeacher λ©”μ„œλ“œ μΆ”κ°€

πŸ”€ 변경사항

πŸ™‹β€β™‚οΈ μ§ˆλ¬Έμ‚¬ν•­

βš—οΈ μ‚¬μš©λ²•

🎸 기타

@Umjiseung Umjiseung added the enhancement New feature or request label Apr 8, 2024
@Umjiseung Umjiseung self-assigned this Apr 8, 2024
@Umjiseung Umjiseung linked an issue Apr 8, 2024 that may be closed by this pull request
νšŒμ›κ°€μž…λ‘œμ§μ— μ„ μƒλ‹˜ κΆŒν•œ λ°”λ‘œ λ„£κΈ° #50
Closed
ta2ye0n
ta2ye0n approved these changes Apr 8, 2024
View reviewed changes
Copy link
Contributor

@ta2ye0n ta2ye0n left a comment

Choose a reason for hiding this comment

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

μˆ˜κ³ ν•˜μ…¨μŠ΅λ‹ˆλ‹€!

@Umjiseung Umjiseung merged commit 9aeb43e into develop Apr 8, 2024
1 check passed
@Umjiseung Umjiseung deleted the 50-signup-teacher-authority-putting branch April 8, 2024 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ta2ye0n ta2ye0n ta2ye0n approved these changes

@enbraining enbraining Awaiting requested review from enbraining

@yusung103 yusung103 Awaiting requested review from yusung103

Assignees

@Umjiseung Umjiseung

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

νšŒμ›κ°€μž…λ‘œμ§μ— μ„ μƒλ‹˜ κΆŒν•œ λ°”λ‘œ λ„£κΈ°
2 participants
@Umjiseung @ta2ye0n