canove / whaticket-community

A very simple Ticket System based on WhatsApp messages, that allow multi-users in same WhatsApp account.
MIT License
1.57k stars 800 forks source link

* use node16 and updated pino-pretty and typescript packages. Fix #483 #487

Closed allgood closed 1 year ago

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

cronoswebdigital commented 1 year ago

Mudei a versão do typescript para 4.1 depois executei os seguintes comandos no backend: npm install npm run build npm r pino-pretty rm -rf dist/ npm i pino-pretty npm run build

allgood commented 1 year ago

Mudei a versão do typescript para 4.1 depois executei os seguintes comandos no backend: npm install npm run build npm r pino-pretty rm -rf dist/ npm i pino-pretty npm run build

o que você fez foi fazer manualmente o que eu coloquei no package.json... ele instala uma versão mais nova do pino-pretty e também instala o arquivo de tipos dele. preferi manter uma versão na mesma série da que já estava em uso para evitar quebra de funcionalidades.

também ajustei o arquivo do docker para usar versão mais nova do node, mas isso é só pra quem vai usar com docker mesmo.

w3nder commented 1 year ago

Talvez atualizando só a lib do pino e mais garantido do que usar o node 16

allgood commented 1 year ago

Talvez atualizando só a lib do pino e mais garantido do que usar o node 16

eu só consegui gerar o container usando node 16, não me lembro agora o erro que deu no 14

allgood commented 1 year ago

adendo:

para funcionar eu precisei fazer três coisas:

possivelmente se não for usar container docker não é necessário usar o node 16, por isso que a alteração é só no dockerfile.

o outro PR tá falhando a build ( #491 ) porque nele eu deixei só a troca do whaticket ... ele vai funcionar depois que este PR for aceito.

allgood commented 1 year ago

Estou cancelando este PR e vou fazer um novo, com uma sequencia de pequenos fixes.

Replaced by PR #493