-
Notifications
You must be signed in to change notification settings - Fork 0
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
62 impl post apis for organization team submodules #63
62 impl post apis for organization team submodules #63
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
007 테스트가 빠진 것 같은데 그것만 다시 확인하고 머지합시다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
example 쭉 만든건 예시용으로 이번에 만든건가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 매번 복사하기도 귀찮고 템플릿으로 만들면 좋을 것 같아서요
DrizzleModule, | ||
SemesterModule, | ||
UserModule, | ||
forwardRef(() => TeamModule), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
오 이거 하마랑 어제 논의했던 내용인데 알아서 잘 썼군
혹시 모듈 설계 순환참조 생겨도 이렇게 한 이유 있나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어쨌든 organization 하위 모듈에서 상위모듈 꺼가 필요해서 발생하는 일이라 순환참조해도 괜찮지 않을까 싶었슴다
/* eslint-disable import/no-cycle */ | ||
import { TeamModule } from "./team/team.module"; | ||
/* eslint-disable import/no-cycle */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이게 위아랫줄이 다 필요했던가..?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
다른 순환참조를 막으려고 일단 한줄에만 걸었어요
ApiOrg007RequestBody, | ||
ApiOrg007RequestUrl, | ||
ApiOrg007ResponseCreated, | ||
ApiOrg008RequestBody, | ||
ApiOrg008RequestUrl, | ||
ApiOrg008ResponseCreated, | ||
ApiOrg009RequestBody, | ||
ApiOrg009RequestUrl, | ||
ApiOrg009ResponseCreated, | ||
apiOrg007, | ||
apiOrg008, | ||
apiOrg009, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 확실히 깔끔하다
|
||
export * from "./endpoint/apiOrg007"; | ||
export { default as apiOrg007 } from "./endpoint/apiOrg007"; // default export 추가 | ||
|
||
export * from "./endpoint/apiOrg008"; | ||
export { default as apiOrg008 } from "./endpoint/apiOrg008"; // default export 추가 | ||
|
||
export * from "./endpoint/apiOrg009"; | ||
export { default as apiOrg009 } from "./endpoint/apiOrg009"; // default export 추가 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 export 두 번 하는 이유가 뭔가요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
위는 requestparam, url, requestbody 같은 것들이고
아래는 default로 나오는 저걸 다묶어서 나오는 api 모듈입니당
원래 클럽스에서는 default로 나오는 거였는데 index로 묶으려면 아랫줄처럼 해줘야 해요 (*로 export 가 안됨)
return res; | ||
} | ||
|
||
async ckTeamMemberBeforeCreate(userId, teamId): Promise<number> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ck가 check인가요?
꼭 필요한거 아니면 약어 안 쓰는게 좋을 것 같긴 해요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아하... convention으로 정리해두긴 했는데 select로 바꿔도 큰 문제 없는 부분이긴 할 것 같습니당
요약 *
It closes #62
Implementation for ApiOrg007~009
스크린샷
apiOrg008
apiOrg009
이후 Task *