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

Remove Selecione Order Filter #125

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

saracamilo
Copy link

@saracamilo saracamilo commented Feb 18, 2025

Q A
Branch? improv/not-show-select-sort-filter
Bug fix? no
New feature? yes
Deprecations? no
Issues Fix #113

COMO ESTÁ

task 113

@saracamilo saracamilo self-assigned this Feb 18, 2025
@saracamilo saracamilo force-pushed the improv/not-show-select-sort-filter branch from 7204794 to 8aa2176 Compare February 18, 2025 16:17
@saracamilo saracamilo marked this pull request as ready for review February 18, 2025 16:20
@saracamilo saracamilo force-pushed the improv/not-show-select-sort-filter branch 6 times, most recently from 0142e8e to 65d84cf Compare February 18, 2025 18:28
@saracamilo
Copy link
Author

Explicando: o php unit dele não estava passando graças ao AuditUpdateListenerTest então pensei e substituí o valor fixo "Hello world!" (que estava sendo apontado pelo erro) por um nome único gerado com uniqid(), pra garantir que sempre uma mudança acontecia. ai o listener de auditoria criaria o novo registro. Não sei se é o caminho certo, mas aguardo revisões.

@alessandrofeitoza
Copy link

Explicando: o php unit dele não estava passando graças ao AuditUpdateListenerTest então pensei e substituí o valor fixo "Hello world!" (que estava sendo apontado pelo erro) por um nome único gerado com uniqid(), pra garantir que sempre uma mudança acontecia. ai o listener de auditoria criaria o novo registro. Não sei se é o caminho certo, mas aguardo revisões.

Sagaz @saracamilo mas acho jogo subir isso em outro PR que seria pra resolver aquela issue #121

@alessandrofeitoza alessandrofeitoza force-pushed the improv/not-show-select-sort-filter branch from 15fc6d7 to d9abf4f Compare February 19, 2025 13:02
@alessandrofeitoza alessandrofeitoza merged commit 2c38180 into main Feb 19, 2025
4 checks passed
@alessandrofeitoza alessandrofeitoza deleted the improv/not-show-select-sort-filter branch February 19, 2025 13:18
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.

[web] Não mostrar a palavra selecione no filtro de ordenação
3 participants