Skip to content
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

Permalink para cadastrar #50

Open
bplmp opened this issue Sep 25, 2018 · 12 comments
Open

Permalink para cadastrar #50

bplmp opened this issue Sep 25, 2018 · 12 comments
Labels
bug Something isn't working hacktoberfest

Comments

@bplmp
Copy link
Contributor

bplmp commented Sep 25, 2018

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.

@bplmp bplmp added the bug Something isn't working label Sep 25, 2018
@ronaiza-cardoso
Copy link
Contributor

@victorsenam está trabalhando nisso?

@ronaiza-cardoso
Copy link
Contributor

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

@bplmp
Copy link
Contributor Author

bplmp commented Oct 19, 2018

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

@ronaiza-cardoso
Copy link
Contributor

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

@ronaiza-cardoso
Copy link
Contributor

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

@bplmp
Copy link
Contributor Author

bplmp commented Oct 19, 2018

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

@ronaiza-cardoso
Copy link
Contributor

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
Copy link
Contributor Author

bplmp commented Oct 19, 2018

Pode, claro.

@victorsenam victorsenam removed their assignment Oct 22, 2018
@victorsenam
Copy link
Contributor

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
Copy link
Contributor

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

é o esperado, não?

@bplmp
Copy link
Contributor Author

bplmp commented Oct 23, 2018

@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
Copy link
Contributor Author

bplmp commented Oct 23, 2018

Resolvido temporariamente o problema do comentário acima. Relacionado a prefeiturasp/SME-fila-da-creche-API#3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hacktoberfest
Projects
None yet
Development

No branches or pull requests

4 participants