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

#436 taxi-sample-generator 병합 #437

Merged
merged 78 commits into from
Jan 23, 2024
Merged

#436 taxi-sample-generator 병합 #437

merged 78 commits into from
Jan 23, 2024

Conversation

kmc7468
Copy link
Member

@kmc7468 kmc7468 commented Nov 21, 2023

Summary

It closes #436
taxi-sample-generator를 taxi-back에 머지하였습니다.
기존 taxi-sample-generator 레포에 남겨진 커밋 로그도 taxi-back에 모두 합쳐졌습니다. (GitHub에선 안보이지만, git blame과 git log를 통해 확인할 수 있어요.)

Further Work

  • Nothing

…r-handle-settlement

#8 change roomschema to better handle settlement
…-with-readfile

Fix: replace readFileSync
src/sampleGenerator/Dockerfile Outdated Show resolved Hide resolved
src/sampleGenerator/README.md Outdated Show resolved Hide resolved
src/sampleGenerator/security.js Outdated Show resolved Hide resolved
@kmc7468 kmc7468 marked this pull request as draft November 27, 2023 02:19
@kmc7468 kmc7468 marked this pull request as ready for review January 6, 2024 12:11
@kmc7468
Copy link
Member Author

kmc7468 commented Jan 6, 2024

앗 tools 쪽 수정을 까먹고 안했네요! 금방 수정하겠읍니다 ㅜ

@kmc7468 kmc7468 marked this pull request as draft January 6, 2024 12:14
@kmc7468
Copy link
Member Author

kmc7468 commented Jan 6, 2024

DB 덤프는 잘 되는 것 같은데, 복원이 잘 안되는 것 같습니다. 혹시 원인이 뭘까요? ㅠㅠ

오류가 뜨는 것도 아니고.. mongodb에 db도 생성되는데 collection들에 데이터가 안들어 있네요.

@kmc7468
Copy link
Member Author

kmc7468 commented Jan 6, 2024

해결했습니다! dump 대신 dump/taxi를 인수로 넘겨주니 잘 작동합니다.

@kmc7468 kmc7468 marked this pull request as ready for review January 6, 2024 12:36
.gitmodules Outdated Show resolved Hide resolved
sampleGenerator Outdated
Copy link
Member

Choose a reason for hiding this comment

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

🎉

@@ -13,7 +13,7 @@
"serve": "cross-env TZ='Asia/Seoul' NODE_ENV=production node app.js",
"runscript": "cross-env TZ='Asia/Seoul' NODE_ENV=production node",
"lint": "npx eslint --fix .",
"sample": "cd sampleGenerator && npm start && cd .."
"sample": "cd src/sampleGenerator && npm start && cd .."
Copy link
Member

Choose a reason for hiding this comment

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

이런식으로 samplegenerator 스크립트 실행시켜도 좋을 것 같네요

https://github.com/sparcs-kaist/taxi-front/blob/0a759afab95386cc5d9663a0e7cd77150f516ab0/package.json#L7

Copy link
Member Author

Choose a reason for hiding this comment

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

pnpm -F taxisamplegenerator 로 시도해 봤는데 안되네요! ㅜㅜ

Copy link
Contributor

@chlehdwon chlehdwon left a comment

Choose a reason for hiding this comment

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

큰 작업 고생 많으셨습니다~!! 너무 잘하셨는데 소소하게 한 가지만 제안 드립니다!

profileImageUrl: generateProfileImageUrl(),
joinat: Date.now(),
subinfo: {
kaist: new String(20230000 + num), // ^-^
Copy link
Contributor

Choose a reason for hiding this comment

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

젊어지는 택시 좋습니다 😁

src/sampleGenerator/.env.example Outdated Show resolved Hide resolved
Copy link
Member

@14KGun 14KGun left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@chlehdwon chlehdwon left a comment

Choose a reason for hiding this comment

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

LGTM!!! 고생많으셨습니다 :)

@kmc7468 kmc7468 merged commit db03037 into dev Jan 23, 2024
1 check passed
@kmc7468 kmc7468 deleted the #436-merge-sample-generator branch January 23, 2024 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚒ enhancement New feature or request testcase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

taxi-sample-generator 병합
6 participants