search

prefeiturasp / SME-FilaDaCreche

GNU Affero General Public License v3.0
35 stars 16 forks source link

Permalink para cadastrar #50

Open bplmp opened 6 years ago

bplmp commented 6 years ago

Atualmente, se você abrir um link direto para a rota do cadastro, aparece uma tela em branco: http://filadacreche.sme.prefeitura.sp.gov.br/#/cadastrar/27

Isso é porque o estado dessa rota vem da rota anterior. Precisamos lidar com isso para permitir um permalink aqui, ou pelo menos redirecionar para outra tela anterior.

ronaiza-cardoso commented 6 years ago

@victorsenam está trabalhando nisso?

ronaiza-cardoso commented 6 years ago

tava pensando em colocar uma espécie de Auth Guard para que não seja possível acessar essa página

bplmp commented 6 years ago

@ronaiza-cardoso a gente não quer bloquear o acesso, mas sim criar um permalink para que possa ser acessada diretamente

ronaiza-cardoso commented 6 years ago

mas ela não funciona sem os dados da página anterior, como seria esse permalink?

ronaiza-cardoso commented 6 years ago

Talvez uma página 404 resolvesse? com redirecionamento para a home

bplmp commented 6 years ago

Redirecionar seria o menos ideal. Um permalink poderia por exemplo ter os parametros da busca nele, como o da rota anterior.

ronaiza-cardoso commented 6 years ago

isso ainda precisa de um pouco de trabalho meu, mas vi que não tem página 404, posso abrir uma issue com o meu pr com ela?

bplmp commented 6 years ago

Pode, claro.

victorsenam commented 6 years ago

Desculpa me deixar de assigned nessa task, estou focando em contribuir no github.com/prefeiturasp/SME-plataforma-curriculo agora. Se puder contribuir com essa eu agradeço.

Eu concordo com o @bplmp, seria bem legal se ela funcionasse que nem a página anterior e eu pudesse copiar a url para compartilhar com alguém ou acessar a qualquer momento. Não é uma mudança muito fácil.

ronaiza-cardoso commented 6 years ago

@victorsenam abri o link que vc enviou e apareceu esse resultado: image

é o esperado, não?

bplmp commented 6 years ago

@ronaiza-cardoso não é o esperado, nem de longe hehe. Está rolando algum problema com nossa API, vou investigar. @ludimila, pra vc saber tmb.

bplmp commented 6 years ago

Resolvido temporariamente o problema do comentário acima. Relacionado a https://github.com/prefeiturasp/SME-fila-da-creche-API/issues/3