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

collection.js를 제거하고 TypeORM으로 리팩토링 #958

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

lshqqytiger
Copy link
Contributor

@lshqqytiger lshqqytiger commented Nov 28, 2021

설명

구버전의 pg 모듈은 최신버전의 Node.js를 지원하지 않고, 최신버전의 pg 모듈은 기존 끄투 코드에서 오작동을 일으킵니다.
따라서 이전 방식의 collection.js를 제거하고 TypeORM을 도입하여 최신버전의 Node.js에서도 끄투를 구동할 수 있도록 하였습니다.
또한 Promise와 async/await으로 대체 가능한 lizard.js를 제거하였습니다.

변경 사항

  • collection.js 제거
  • lizard.js 제거
  • TypeORM 추가
  • Promise 추가
  • kkutu_manner_ko, kkutu_manner_en 테이블의 처리 방식 변경(columns -> json)
  • Redis 버전업 & 처리 방식 변경
  • 기타 등등..

적용 시 주의 사항 및 피드백 요청 사항

  • 매우 불안정할 수 있습니다. 기존 프리서버에 적용하기보다 새로 프리서버를 구축할 때, 구버전(v12)의 Node.js를 사용할 수 없는 환경에서 사용하는 것을 권장합니다. (이 PR을 기존 프리서버에 synchronized 상태로 적용하려면 기존 DB를 갈아엎고 새로 구축하거나 약간의 노가다를 해야합니다)
  • 문제 발생 시 comment 남겨주세요.

Remove collection
Remove lizard
Add TypeORM
Add Promise
Support latest version of Node.js
The foot of the candle is dark.
@ghost
Copy link

ghost commented Nov 28, 2021

image
타자 대결을 할시, 일부 단어가 중복이 됩니다. 그리고 게임이 다 끝난후 BGM이 안꺼집니다.

@lshqqytiger
Copy link
Contributor Author

두 문제 모두 본 PR과는 큰 연관성이 없는 것 같습니다.

README.md Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Jan 1, 2022

image
공지를 띄울때 메세지가 남겨집니다.

@lshqqytiger lshqqytiger changed the title Support Node.js v17 & Add TypeORM collection.js를 제거하고 TypeORM으로 리팩토링 Jul 31, 2022
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.

2 participants