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

Data e hora do evento não salva corretamente - R$ 80,00 #36

Closed
alexanmtz opened this Issue Oct 28, 2017 · 8 comments

Comments

2 participants
@alexanmtz
Collaborator

alexanmtz commented Oct 28, 2017

Os eventos são criados usando um calendário em Javascript que escreve num campo hidden de data e hora o que foi escolhido pelo usuário para início e fim do evento. Porém na hora de exibir o evento está aparecendo com datas e horários divergentes.

Isto acontece devido a unificação de data entre o javascript e o ruby ser necessária. Tentei realizar as conversões colocando todos em UTC mas mesmo assim não deu certo. Sendo assim, para solucionar este problema pode ser preciso trabalhar com timezones.

Como chegar no cenário descrito

  • Criar uma conta como guia
  • Ir em Criar truppie
  • Escolher as datas de saída e retorno
  • Visualizar evento
  • As datas dos eventos não correspondem sempre a data correta

Critério de aceitação:

  • A data escolhida na hora de cadastrar o evento sempre corresponder de forma correta a data que aparece na visualização, bem como salvar a data corretamente no banco.

Considerações sobre a plataforma:

  • Existe um javascript de calendário que irá se transformar num componente, mas no momento ele é um script na página com algumas limitações.
  • As datas escolhidas são salvas em campos para serem salvas no servidor quando o evento é salvo pelo guia.

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

Como ter um pull request aceito e integrado da forma correta no projeto

Para que seu Pull Request seja aceito sem maiores problemas e integrado diretamente para que possa ir para produção você deve sempre estar integrado com o master do projeto, sempre usando rebase como padrão.

Se tiver dúvidas como fazer isto dá uma olhada aqui neste post

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 👍 o

@alexanmtz alexanmtz added the bug label Oct 28, 2017

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

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Oct 28, 2017

@3fernandez

This comment has been minimized.

Contributor

3fernandez commented Oct 28, 2017

@alexanmtz posso dar uma olhada nisto?

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

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Oct 28, 2017

Pode pegar sim @3fernandez, esta tarefa é um pouco urgente, pois é um bug grave que atrapalha as datas dos eventos e que já quebrei bastante a cabeça por isto, então prefiro que pegue ela antes de qualquer outra.

O componente de calendário que criei para ele está um pouco bagunçado, mas vou extrai-lo num componente separado aqui: https://github.com/alexanmtz/trip-range. Se tiver interesse poderá contribuir posteriormente.

Qualquer dúvida é só falar.

@3fernandez

This comment has been minimized.

Contributor

3fernandez commented Oct 28, 2017

Show de bola, vou priorizar.

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Oct 28, 2017

Não esqueça desta vez @3fernandez que sempre que obter as atualizações do projeto usar sempre rebase a partir da sua branch, para não termos maiores dores de cabeça na integração. Infelizmente por conta de um merge eu perdi suas indentações :-(

@3fernandez

This comment has been minimized.

Contributor

3fernandez commented Oct 28, 2017

Aah poxxa 😞... Blza então.

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

@alexanmtz

This comment has been minimized.

Collaborator

alexanmtz commented Nov 17, 2017

Comprovante de transferência enviado e código já integrado como foi conversado @3fernandez . Obrigado pela contribuição!

@alexanmtz alexanmtz closed this Nov 17, 2017

@3fernandez

This comment has been minimized.

Contributor

3fernandez commented Nov 18, 2017

Opa @alexanmtz, muito obrigado. Está sendo legal pegar essas issues e discutir soluções consigo, estarei atento pra pegar mais. 🤘

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

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