-
Notifications
You must be signed in to change notification settings - Fork 2
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
[12.0][ADD] project_status_sla: add new module #20
Conversation
f8aaf34
to
23b3dc9
Compare
@marcelsavegnago @WesleyOliveira98 PR liberada para revisão, lembrando que fiz a alteração para considerar a questão que o Cliente indicou no chamado, onde ele necessita de um SLA referente ao tempo em que o projeto deve ficar em um status especifico |
23b3dc9
to
b975221
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Foi validado com o cliente.. vamo subir em staging ou prod ? |
Acredito que inicialmente podemos subir na Staging, o meu ambiente do Doodba na v12 esta dando algum conflito que o banco nao sobe, nesse tempo tentando arrumar acabei não conseguindo reunir com o cliente para validar e demonstrar, assim que o pessoal der o ok, simular o cotidiano deles ai acredito que podemos prosseguir para Prod |
mas precisa validar algo antes de enviar para a staging deles ? |
O ultimo teste que fiz, estava ok, a validação de quando um projeto estava em um estagio e era criado o SLA para ele estava contabilizando corretamente, faltou eu testar em grande quantidade de SLA para ver se todas notificações chegariam corretamente, vou buscar uma solução para o build do meu Doodba na v12 para que eu consiga subir o banco, atualmente ele fica em loop |
b975221
to
9b19835
Compare
9b19835
to
5e7007e
Compare
@marcelsavegnago Testes finalizados, correções e refatorações aplicadas, incluindo testes unitários também e métodos de _prepare para facilitar em futuras migrações |
cc @marcelsavegnago @WesleyOliveira98