Finalizando a estrutura de tarefas agendadas e entrega da estória 24, lembrete choosedate doador.

SharebookBR / sharebook-backend

Projeto backend de código livre para o app Sharebook.

117 stars 75 forks source link

Finalizando a estrutura de tarefas agendadas e entrega da estória 24, lembrete choosedate doador. #190

Closed raffacabofrio closed 5 years ago

raffacabofrio commented 5 years ago

Salve senhores,

Gostaria de pedir um cuidado especial nesse code review. Trata-se de uma estrutura nova que a gente ainda não tinha até agora. TAREFAS AGENDADAS. Podem criticar mesmo. Que eu faço os ajustes até todos nós ficarmos confiantes. Tem que ficar top essa porra!

Minha intenção foi criar uma estrutura que seja simples, fácil de usar, segura e que poupe recursos do nosso servidor compartilhado baratex. :-)

Vocês vão ver que criar uma novo job vai ser molezinha. Só criar a classe e implementar o método Work(). Suave!

Enfim. Bora pro code review!

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.20-xgvcjdnw completed (commit https://github.com/SharebookBR/sharebook-backend/commit/b8008241fe by @raffacabofrio)

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.20-yloiunnw completed (commit https://github.com/SharebookBR/sharebook-backend/commit/e37c3d27d6 by @raffasharecare)

raffacabofrio commented 5 years ago

@danielaloisio e @walter-lopes ,

Fiz todos os ajustes que vcs pediram. Se tiver mais pode falar heim que hoje eu estou nervoso!!!

bhenn commented 5 years ago

Tem alguma regra específica para o nome dos arquivos terem ficado assim ?

raffacabofrio commented 5 years ago

Salve @bhenn

Excelente ideia mano. Realmente receber e-mail na madrugada seria estranho e menos eficiente. Vou criar um atributo, talvez com nome de BestExecutionHour em que o executor tente se aproximar. Muito bom!

A ordem de execução dos Jobs tem uma grande importância. Eu optei por colocar esse número no nome do arquivo pra dar essa visibilidade imediata para os novos devs. Por exemplo o job 1 notifica o doador que eh o dia da decisão. O job 3 remove o livro da vitrine no dia da decisão. Caso o job3 rodasse antes do job1 esse não encontraria nenhum registro e não faria seu trabalho. Sacou?

Ah, e muito obrigado pelo feedback mano.

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.23-ldcwyfko completed (commit https://github.com/SharebookBR/sharebook-backend/commit/302a71f6ff by @raffasharecare)

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.23-bpmqwlec completed (commit https://github.com/SharebookBR/sharebook-backend/commit/bb5f79917a by @raffacabofrio)

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.23-jrxfjkbk completed (commit https://github.com/SharebookBR/sharebook-backend/commit/6ef1f7ab18 by @raffacabofrio)

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.23-peldhogi completed (commit https://github.com/SharebookBR/sharebook-backend/commit/6b9acfbd0e by @raffacabofrio)

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.23-nqcrttrx completed (commit https://github.com/SharebookBR/sharebook-backend/commit/3ee64e54df by @raffacabofrio)

raffacabofrio commented 5 years ago

Salve @felippem fiz todos os ajustes que vc pediu. E também fiz alguns comentários. Tmj