search

ciandt / tech-gallery

A living Technology Gallery built on top of Google App Engine's platform.
Apache License 2.0
39 stars 31 forks source link

Fixing when Tecnology ID contains '/' causing router problems #188

Closed tiago-marques closed 8 years ago

tiago-marques commented 8 years ago

Fixing when Tecnology ID contains '/' causing router problems and fixing misspelled comment

tiago-marques commented 8 years ago

No Angular ao carregar uma tecnologia com um ID que contenha '/' acaba causando problemas de rota. (https://techgallery.ciandt.com/#/technologies/google_drive_api_/_gda). O Angular considera o '/' como uma outra rota.

Corrigindo o problema ao criar uma tecnologia com '/' , muito comum quando a tecnologia é reconhecida por dois nomes.

Ex. React / ReactJS, Java Server Facets / JSF

Esse pull-request evita o mesmo problema no futuro. Precisando atualizar os que tenham esse problema no database.

tiago-marques commented 8 years ago

Considerar o pull-request.

ltogniolli commented 8 years ago

essa é a issue #183

será que # é o mesmo caso e já dá pra tratar junto?

tiago-marques commented 8 years ago

Conforme orientação do @ltogniolli estou fechando esse pull-request para atender a issue por completo. Novo pull-request #190

brunocarneiro commented 8 years ago

It is not working. I deployed on develop server and tested. We should investigate in more details. Frontend is sending the correct id, however backend is regenerating id wrongly.

brunocarneiro commented 8 years ago

I think our SCM model is not prepared to work as open source as I haven't found any way to reopen this bug as it is already merged. However not in our production branch. @ltogniolli @larruda