Skip to content

(refactor) FindAll method ignore accent in search #143

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

Open
RamonJales opened this issue Jan 11, 2024 · 3 comments · May be fixed by #183
Open

(refactor) FindAll method ignore accent in search #143

RamonJales opened this issue Jan 11, 2024 · 3 comments · May be fixed by #183
Labels
enhancement New feature or request

Comments

@RamonJales
Copy link
Collaborator

Como é o trecho de código atualmente?

Atualmente os métodos findAll por um partialName levam em consideração os acentos.

Como deve ser?

Não devem levar em consideração os acentos, deve pesquisar mesmo assim.

@Kywal
Copy link
Collaborator

Kywal commented Jan 11, 2024

lembra de ajustar o título do issue

@isaacmsl
Copy link
Owner

isaacmsl commented Jan 12, 2024

@RamonJales , o título da issue deve ser sugestivo com o quê se pretende refatorar.
Vou retirar o status de invalid assim que for resolvido esse problema. A ideia da issue julgo importante.

@isaacmsl isaacmsl added the invalid This doesn't seem right label Jan 12, 2024
@RamonJales
Copy link
Collaborator Author

Eu tinha colocado o nome certo. Pensei que você tivesse editado ou Kywal.

@RamonJales RamonJales changed the title (refactor) Proponho uma refatoração! (refactor) FindAll method ignore accent in search Jan 12, 2024
@Kywal Kywal added enhancement New feature or request and removed invalid This doesn't seem right labels Jan 13, 2024
@RamonJales RamonJales linked a pull request Jan 18, 2024 that will close this issue
5 tasks
@RamonJales RamonJales linked a pull request Jan 18, 2024 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

3 participants