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

feat(be): add course #2357

Draft
wants to merge 12 commits into
base: main
Choose a base branch
from
Draft

feat(be): add course #2357

wants to merge 12 commits into from

Conversation

w0uldy0u
Copy link
Contributor

@w0uldy0u w0uldy0u commented Feb 8, 2025

Description

Course 관련 기능들 개발
(CRUD, join, leave)

Additional context

많은 질문과.. 토론 끝에 모두 client 쪽에서 처리하는걸로 결정했습니다.
프론트 측에서도 client(user)용 management 페이지를 따로 만드는 것으로 얘기해서 이렇게 결정했습니다.

Management 페이지를 통합하고 해당 API들은 다 admin쪽에 개발합니다

group이 course와 study 두개로만 쓰인다면 Course, Study 둘 다 공통으로 동일 로직을 사용하는 메서드를 제외하고는 group에서 정리해야되지 않을까...? 싶습니다.


Closes TAS-1248

Before submitting the PR, please make sure you do the following

Copy link

@coolify-skkuding
Copy link

coolify-skkuding bot commented Feb 8, 2025

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2025-02-13 17:05:42 CET

@w0uldy0u w0uldy0u changed the title feat(be): add course feat(be): add course Feb 8, 2025
@w0uldy0u w0uldy0u self-assigned this Feb 9, 2025
@Jaehyeon1020
Copy link
Member

Contact에 지금 필드가 4개정도인 것 같은데, JSON 타입 말고 따로 모델을 두는 건 어떨까요? 후에 연락처 이메일 검색 기능이 추가되거나 정보 수정이 필요할 때 모델이 따로 있는 편이 좋을 것 같습니다

@w0uldy0u
Copy link
Contributor Author

Contact에 지금 필드가 4개정도인 것 같은데, JSON 타입 말고 따로 모델을 두는 건 어떨까요? 후에 연락처 이메일 검색 기능이 추가되거나 정보 수정이 필요할 때 모델이 따로 있는 편이 좋을 것 같습니다

넵 그렇게 수정하겠습니다

Copy link

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.

2 participants