12.06.2018, 13:37
Cara, a ideia й muito boa, mas a execuзгo й muito mais ou menos (com o mais para menos).
Vocк nгo seguiu os padrхes de desenvolvimento de um app nodejs, o mais simples nгo forneceu: package.json. As pessoas nгo tкm que ficar escrevendo npm install isso e aquilo, elas sу tкm que escrever npm install e pronto.
Editar a source para configurar a conta do gmail tambйm й outra falha da estrutura, coloca as configuraзхes num arquivo json e carrega de lб.
Vocк faz tudo manualmente usando http e url, fez seu prуprio modo de extrair os dados. Isso й terrнvel!
Vocк nгo verifica se o email й vбlido ou nгo antes de enviб-lo, talvez o nodemailer faзa, mas й algo tгo simples que dб para vocк fazer antes e retornar um erro especificando.
Vocк cria um transporter do nodemailer a cada requisiзгo http, isso й loucura cara!
Por fim, pode haver outros problemas na aplicaзгo, mas temos que ir por partes porque vocк deve estar aprendendo ainda e deve receber as crнticas com calma.
Sу gostaria que dividisse sua release em dois repositуrios, um contendo o conteъdo javascript e outro com o conteъdo pawn, separando a API da interface de utilizaзгo, saca?
Recomendo que faзa assim (coloquei mailer porque й o nome mais gerйnico para coisas desse tipo, mas fica a teu critйrio):
samp-mailer-api -> Repositуrio da API em nodejs
samp-mailer -> Repositуrio da include
O projeto fica melhor assim, mais organizado. Dб para gente colocar atй suporte a sampctl na tua include de boa (posso fazer isso por vocк, jб que muita gente nessa board nгo o conhece).
Bom, qualquer coisa eu dou um fork no teu repo e ajudo na refatoraзгo do cуdigo. Boa sorte!
Vocк nгo seguiu os padrхes de desenvolvimento de um app nodejs, o mais simples nгo forneceu: package.json. As pessoas nгo tкm que ficar escrevendo npm install isso e aquilo, elas sу tкm que escrever npm install e pronto.
Editar a source para configurar a conta do gmail tambйm й outra falha da estrutura, coloca as configuraзхes num arquivo json e carrega de lб.
Vocк faz tudo manualmente usando http e url, fez seu prуprio modo de extrair os dados. Isso й terrнvel!
Vocк nгo verifica se o email й vбlido ou nгo antes de enviб-lo, talvez o nodemailer faзa, mas й algo tгo simples que dб para vocк fazer antes e retornar um erro especificando.
Vocк cria um transporter do nodemailer a cada requisiзгo http, isso й loucura cara!
Por fim, pode haver outros problemas na aplicaзгo, mas temos que ir por partes porque vocк deve estar aprendendo ainda e deve receber as crнticas com calma.
Sу gostaria que dividisse sua release em dois repositуrios, um contendo o conteъdo javascript e outro com o conteъdo pawn, separando a API da interface de utilizaзгo, saca?
Recomendo que faзa assim (coloquei mailer porque й o nome mais gerйnico para coisas desse tipo, mas fica a teu critйrio):
samp-mailer-api -> Repositуrio da API em nodejs
samp-mailer -> Repositуrio da include
O projeto fica melhor assim, mais organizado. Dб para gente colocar atй suporte a sampctl na tua include de boa (posso fazer isso por vocк, jб que muita gente nessa board nгo o conhece).
Bom, qualquer coisa eu dou um fork no teu repo e ajudo na refatoraзгo do cуdigo. Boa sorte!