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

fix(page-job-scheduler): corrige erro quando 'firstExecution' undefined #2327

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

anderson-gregorio-totvs
Copy link
Contributor

Corrigo erro quando 'firstExecution' é definido como undefined

fixes DTHFUI-10385

page-job-scheduler

DTHFUI-10385


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
Ao definir firstExecution como undefined um erro é apresentado no console

Qual o novo comportamento?
Ao definir firstExecution como undefined não deve ser apresentado no console

Simulação
DTHFUI-10385.zip

@anderson-gregorio-totvs anderson-gregorio-totvs force-pushed the job-scheduler/DTHFUI-10385-v17 branch from ddc65cb to 0aea5f7 Compare December 5, 2024 22:28
Corrigo erro quando 'firstExecution' é definido como undefined

fixes DTHFUI-10385
@anderson-gregorio-totvs anderson-gregorio-totvs force-pushed the job-scheduler/DTHFUI-10385-v17 branch from 0aea5f7 to 78286f3 Compare December 6, 2024 15:13
@pedrodominguesp pedrodominguesp merged commit b32312f into 17.x.x Dec 6, 2024
8 checks passed
@pedrodominguesp pedrodominguesp deleted the job-scheduler/DTHFUI-10385-v17 branch December 6, 2024 20:05
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.

3 participants