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

62 impl post apis for organization team submodules #63

Merged
merged 5 commits into from
Nov 29, 2024

Conversation

Gerbera3090
Copy link
Contributor

@Gerbera3090 Gerbera3090 commented Nov 29, 2024

요약 *

It closes #62
Implementation for ApiOrg007~009

스크린샷

apiOrg008
Pasted Graphic 3
Pasted Graphic 2
image

apiOrg009
Pasted Graphic 4
Pasted Graphic 5
Pasted Graphic 6

이후 Task *

  • PRP 관련 POST 및 GET API 생성

@Gerbera3090 Gerbera3090 linked an issue Nov 29, 2024 that may be closed by this pull request
3 tasks
Copy link
Contributor

@wjeongchoi wjeongchoi left a comment

Choose a reason for hiding this comment

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

007 테스트가 빠진 것 같은데 그것만 다시 확인하고 머지합시다

Copy link
Contributor

Choose a reason for hiding this comment

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

example 쭉 만든건 예시용으로 이번에 만든건가요?

Copy link
Contributor Author

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),
Copy link
Contributor

Choose a reason for hiding this comment

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

오 이거 하마랑 어제 논의했던 내용인데 알아서 잘 썼군
혹시 모듈 설계 순환참조 생겨도 이렇게 한 이유 있나요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

어쨌든 organization 하위 모듈에서 상위모듈 꺼가 필요해서 발생하는 일이라 순환참조해도 괜찮지 않을까 싶었슴다

Comment on lines +10 to +12
/* eslint-disable import/no-cycle */
import { TeamModule } from "./team/team.module";
/* eslint-disable import/no-cycle */
Copy link
Contributor

Choose a reason for hiding this comment

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

이게 위아랫줄이 다 필요했던가..?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

다른 순환참조를 막으려고 일단 한줄에만 걸었어요

Comment on lines +3 to +14
ApiOrg007RequestBody,
ApiOrg007RequestUrl,
ApiOrg007ResponseCreated,
ApiOrg008RequestBody,
ApiOrg008RequestUrl,
ApiOrg008ResponseCreated,
ApiOrg009RequestBody,
ApiOrg009RequestUrl,
ApiOrg009ResponseCreated,
apiOrg007,
apiOrg008,
apiOrg009,
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 확실히 깔끔하다

Comment on lines +18 to +26

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 추가
Copy link
Contributor

Choose a reason for hiding this comment

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

이거 export 두 번 하는 이유가 뭔가요

Copy link
Contributor Author

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> {
Copy link
Contributor

Choose a reason for hiding this comment

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

ck가 check인가요?
꼭 필요한거 아니면 약어 안 쓰는게 좋을 것 같긴 해요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아하... convention으로 정리해두긴 했는데 select로 바꿔도 큰 문제 없는 부분이긴 할 것 같습니당

@Gerbera3090 Gerbera3090 merged commit ccf5e2f into dev Nov 29, 2024
1 check passed
@Gerbera3090 Gerbera3090 self-assigned this Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Impl post apis for organization team submodules
2 participants