Open
Conversation
ronifabio
requested changes
May 9, 2023
Contributor
ronifabio
left a comment
ronifabio
left a comment
There was a problem hiding this comment.
- Pelo que percebi, está listando todos os profissionais no autocomplete, se tivessemos 1 milhão de usuários, seria inapropriado. Listar somente após o usuário digitar algo. Uma dica é fazer a requisição ao servidor via ajax a cada 3 caracteres digitados.
- Remover o jquery-UI e usar o autocomplete do materialize: https://materializecss.com/autocomplete.html
- Não está funcionando para email no autocomplete. A dica é verificar se tem o @, se tiver, manda a requisição para a rota que lista os nomes a partir do email e mostra o email também no option. Exemplo: Roni Fabio Banaszewski (ronifabio@gmail.com)
Contributor
There was a problem hiding this comment.
Verificar se este método está sendo usado. Contém nomes estáticos. Está estranho.
|
|
||
| mv.addObject("id", id.orElse(0L)); | ||
|
|
||
| List<User> professionals = userService.findProfessionalsNotExist(); |
Contributor
There was a problem hiding this comment.
Está buscando todos os profissionais, não é o correto para o auto-complete. Imagine se tiver 100k de usuários. Deveria usar um like do SQL para buscar de acordo com os termos digitados. Limitar, trazer apenas umas 5 sugestões.
| <link href="https://fonts.googleapis.com/icon?family=Material+Icons" rel="stylesheet"> | ||
| <link href="assets/libraries/materialize/css/materialize.min.css" type="text/css" rel="stylesheet" media="screen,projection"/> | ||
| <link href="assets/resources/styles/style.css" type="text/css" rel="stylesheet" media="screen,projection"/> | ||
| <link href="https://cdnjs.cloudflare.com/ajax/libs/select2/4.0.6-rc.0/css/select2.min.css" rel="stylesheet" /> |
Contributor
There was a problem hiding this comment.
Usar o autocomplete do materialize
Contributor
There was a problem hiding this comment.
Usar id no estilo dashed-case, exemplo: copy-link
…o feature/add_professionals_company
ronifabio
requested changes
Jun 15, 2023
Contributor
ronifabio
left a comment
ronifabio
left a comment
There was a problem hiding this comment.
- Não está funcionando corretamente.
- A empresa 5 tem apenas um funcionário, que é o usuário 6. Porém, é apresentada uma lista com 3 funcionários.
- Ademais, esta lista só aparece depois de tentar adicionar um funcionário. Deveria aparecer antes mesmo de adicionar.
- No modal de adição de funcionário, ao invés de aparecer o botão com nome "Salvar", colocar o botão com texto "Adicionar à empresa". Mas só habilitar o botão quando for um email válido.
- Alinhas os dois botões à direita
- Ao adicionar, mostrar o cartão do usuário convidado, mostrando que está pendente, similar como ocorre no GitHub
- A remoção de um funcionário via clique no ícone de lixeira do cartão também não está funcionando.
ronifabio
requested changes
Jun 21, 2023
Contributor
ronifabio
left a comment
ronifabio
left a comment
There was a problem hiding this comment.
- Não está apresentando o estado de pendente para um funcionário tal como está no exemplo do GitHub. Ao adicionar um funcionário, mesmo que precise de confirmação dele, mesmo que ele não tenha cadastro, apresentar logo na lista de funcionários, mas com uma indicação de que está pendente.
- Também apresentar uma mensagem de feedback que o funcionário foi convidado por email e está no aguardo da confirmação.
- Fazer verificação para poder adicionar apenas funcionários do tipo USER, não permitir adicionar funcionários do tipo COMPANY.
- Também verificar se o funcionário já não está adicionado, para evitar duplicidade.
| mv.addObject("expertises", expertiseDTOs); | ||
| mv.addObject("userInfo", individualInfo); | ||
| // mv.addObject("expertises", professionDTOs); | ||
| mv.addObject("individualInfo", individualInfo); |
Contributor
There was a problem hiding this comment.
- Verifique que agora é userInfo, para servir tanto para individual como para company.
- Porém, este objeto já está sendo setado no TemplateInfoFilter, assim, não precisa passar no controller.
- O statisticInfo também é importante, parece que você removeu. Contém as estatísticas para apresentar no painel lateral do profissional e empresa.
ronifabio
requested changes
Oct 19, 2023
Contributor
ronifabio
left a comment
ronifabio
left a comment
There was a problem hiding this comment.
- Resolver os conflitos.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.