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

Na página de adicionar truppie, os campos que aceitam tags submetem o formulário - R$ 60,00 #28

Closed
alexanmtz opened this Issue Oct 18, 2017 · 11 comments

Comments

3 participants
@alexanmtz
Collaborator

alexanmtz commented Oct 18, 2017

Usamos o taginput (ótimo plugin para jQuery) para inserir tags de itens (como tags, o que inclui num evento etc), e ele funcionava bem até então. Agora, no momento em que os dados de um tour (que chamamos de truppie) é preenchido, as tags quando inseridas e pressionado Enter, o formulário é enviado. O plugin deveria evitar a submissão do formulário. Isto de fato acontece em outros lugares onde este campo está presente, como na adminstração de um tour (tours -> edit)

screen shot 2017-10-18 at 11 25 38 pm

Como chegar no cenário descrito

  • Criar um usuário
  • Criar um perfil de guia no item do menu principal Sou guia
  • Com o seu perfil de guia, ir em Truppies -> criar truppie
  • Tentar preencher tags no campo de tags (assim como outros campos onde o tagsinput é aplicado) e verificar que o formulário é enviado.

Critério de aceitação:

  • Não enviar o formulário com os dados quando apertamos enter ao inserir uma nova tag no campo

Considerações sobre a plataforma:

  • A plataforma tem vários lugares com esse campo e somente nesta seção o formulário está sendo enviado de forma indevida

Requisitos para rodar a plataforma

  • Ruby
  • Postgres
  • Se tornar administrador no sistema
    (para isto, basta criar um arquivo .env com os campos:
 ADMIN_EMAIL=seuemail 
 ADMIN_EMAIL_ALT=outroemail

Como realizar esta tarefa

  1. Comente aqui manifestando o interesse em realizar esta tarefa, que entrarei em contato para obter os detalhes para realizar o pagamento
  2. Faça um fork do projeto
  3. Faça um clone do projeto para o seu ambiente local

rake test (rodar os testes automatizados)

rails db:migrate (Rodar as migrations para o banco de dados)

rails s (Rodar o projeto)

  1. Realize as modificações como descrito nesta Issue
  2. Enviar um Pull Request
  3. Certificar-se de que as modificações passaram nos testes automatizados
  4. Um novo ambiente para testes será criado para validação das modificações com a nossa equipe
  5. Receber os comentários e feedbacks sobre modificações
  6. Enviar quaisquer modificações necessárias até ser aprovado
  7. Você ganha então um 👍
  8. Quando o Pull Request for finalizado, o pagamento é enviado

Se precisar de qualquer ajuda ou tiver qualquer dúvida basta comentar aqui que as dúvidas poderão ser respondidas por qualquer pessoa da comunidade ou quem contribui com o projeto e principalmente eu 👍

Se quiser se familiarizar com o funcionamento de projetos de software livre, tem este excelente guia do Tableless

@alexanmtz alexanmtz added the bug label Oct 18, 2017

@alexanmtz alexanmtz added this to the 3.0 milestone Oct 18, 2017

@alexanmtz alexanmtz added this to Backlog in truppie Oct 19, 2017

@jcserracampos

This comment has been minimized.

jcserracampos commented Oct 19, 2017

Posso ficar com esta?

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Oct 19, 2017

@jcserracampos , pode sim! Eu tentei várias possibilidades e nenhuma funcional, vai ser muito bom ter uma outra visão, qualquer dúvida é só falar!

@alexanmtz alexanmtz moved this from Backlog to In Progress in truppie Oct 22, 2017

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Oct 24, 2017

Olá @jcserracampos , algum progresso?

@jcserracampos

This comment has been minimized.

jcserracampos commented Oct 26, 2017

@alexanmtz Desculpa a demora. Estou terminando de checar umas possibilidades. Até amanhã eu devo fazer o PR.

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Oct 30, 2017

e ai @jcserracampos, alguma luz? 🤞

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Nov 2, 2017

Vou disponibilizar esta tarefa novamente, pois é um bug em produção que precisa ser corrigido. Então quem quiser encarar se manifeste 🔢

alexanmtz added a commit that referenced this issue Nov 2, 2017

@tauanlemos

This comment has been minimized.

Contributor

tauanlemos commented Nov 2, 2017

Posso pegar essa?

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Nov 3, 2017

sim @tauanlemos ! Me dê um retorno assim que tiver uma luz.

@tauanlemos

This comment has been minimized.

Contributor

tauanlemos commented Nov 4, 2017

Você pode me confirmar só quais outras áreas usam o plugin? Só para razões de teste. A funcionalidade de EDIT tb não estava prevenindo o submit do form, acho que resolvi a issue mas só quero garantir que tá funcionando nas outras partes do sistema.

Obs: Não consegui rodar os testes aqui no meu ambiente local

@tauanlemos tauanlemos referenced this issue Nov 4, 2017

Merged

Fix issue #28 #45

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Nov 6, 2017

olá @tauanlemos , vou testar sua PR, muito obrigado pela rápida resposta!

Para o usuário final ela é usada somente na página de edição e criação de um evento, em outras áreas ela é usada pelos admins do sistema quando precisam criar / editar eventos manualmente e estava funcionando, isto que era estranho.

as rotas são edit_guided_tour e guided_tour

@alexanmtz alexanmtz closed this in #45 Nov 6, 2017

alexanmtz added a commit that referenced this issue Nov 6, 2017

Fix issue #28 (#45)
bug that submit the form when inserting tags
@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Nov 6, 2017

Sua PR foi integrada @tauanlemos , excelente trabalho! Em alguns minutos seu código entrará em produção e mais um bug resolvido!

Por favor envie um e-mail para ola@truppie.com com os dados bancários para transferência

@alexanmtz alexanmtz moved this from In Progress to Done in truppie Nov 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment