search

worknenjoy / truppie

The marketplace for adventures
http://www.truppie.com
MIT License
51 stars 38 forks source link

Exibe roteiro com tour #132

Closed amandaclarck closed 6 years ago

amandaclarck commented 6 years ago
amandaclarck commented 6 years ago

@alexanmtz Olá, tudo bem?

Pode verificar se está correto ou precisa de alguma alteração, por favor?

alexanmtz commented 6 years ago

@amandaclarck , obrigado por ter enviado esta PR tão rápido. Acho que algumas partes ainda não estão claras, se você observar, esta página (como mando em anexo) é da gerência de tours e guidebooks por parte do guia, e o template correto para ele está no organizer

O formato da url seria https://truppie.com/organizers/:organizer/manage/:tour

Mas terá que ser modificado para https://truppie.com/organizers/:organizer/manage/tour/:tour

Na barra lateral temos os tours e as reservas realizadas para um tour, mas que deveria lista de cada um.

Nesta mesma barra devemos listar guidebooks, seguindo exatamente o mesmo modelo, e quando clicamos em um deles vai para a página de um guidebook com suas reservas.

O formato da url seria https://truppie.com/organizers/:organizer/manage/:guidebook

Mas terá de ser modificado para https://truppie.com/organizers/:organizer/manage/guidebook/:guidebook

Então você precisa mexer no template que lista na lateral para listar as reservas destes dois modelos, com a possibilidade de visualizar cada um separadamente. E um modelo de view para guidebooks que vai ser praticamente igual ao modelo da rota manage para um tour.

Você já fez um template com a idéia, o que vai ajudar a reaproveitar o código. Este template poderia também ser tanto para tour quanto para guidebook, pois as reservas terão a mesma informação de um order, pois ambos usam ele como modelo para uma reserva.

Creio então que os passos sejam

  1. Alterar a rota atual para que https://truppie.com/organizers/:organizer/manage/:tour seja https://truppie.com/organizers/:organizer/manage/tour/:tour para que possamos separar a listagem de reservas, tanto para um quanto para outro sem colisão entre ambos (caso contrário não teremos como distinguir)
  2. Esta mudança de rota irá afetar o link "Reservas", que deverá direcionar por padrão para o primeiro tour (isto já acontece hoje, mas o link vai para manage sem saber se é um tour ou roteiro)
  3. No menu lateral onde lista os tours para consultar as reservas, deveremos ter duas listas, uma com Tours guiados e outra com Roteiros personalizados com o thumbnail com o link para aquele tour / roteiro.
  4. Clicando em cada um ele carrega o template com as reservas de cada um, que são os orders relacionados
  5. Tente minimizar a duplicação e reutilizar templates, pois principalmente neste caso, as listagens serão praticamente iguais, mas de modelos diferentes

screenshot-truppie

alexanmtz commented 6 years ago

Outra questão @amandaclarck que vale mencionar separadamente, é que se você observar, sua PR veio também com as modificações do teste, que já foram aprovadas. Você precisa sincronizar seu fork com o master e a partir daí criar uma branch. Lembre-se dos passos que já passamos para realizar isto, e se necessário crie um novo PR ou dê um push usando -f para forçar o envio para esta branch para ficar de acordo com o novo master.

O seu fork não está em dia com o master. Sua mudança entrou no master e está vindo aqui novamente, o que atrapalha o merge. Verifique isto por favor e você pode confirmar isto quando enviar o seu PR, observando que só as mudanças relativas a esta PR devem vir.

Qualquer dúvida é só falar e segue o baile 👩‍💻