feat: add PetsRs link to shelter card

coelhotatiane commented 4 months ago

OBJETIVO DO PR Adição de uma nova feat. Task aberta aqui

O QUE CONSTA NO PR

[x] Cada abrigo de pet deve ter um link visível que redirecione para a página correspondente no site Pets RS. Observações: não há um id mútuo entre os dois bancos (sos vs pets rs). A busca pelo abrigo correspondente na API de destino está sendo realizada utilizando o nome do abrigo da seguinte forma: 1. O nome do abrigo aberto pelo usuário é selecionado; 2. Grande parte dos abrigos possui algum separador no nome com algum comentário que não faz parte do nome. Exemplo: "Escola Estadual Infante Dom Henrique [exclusivo Pets]". Por conta disso, tudo que vier depois de separadores como (, -, [ e { será ignorado. Neste caso, a busca será feita considerando "Escola Estadual Infante Dom Henrique". Há possíveis melhorias, mas esse é um start para deixar a busca pelo abrigo pet correspondente mais eficiente. 3. Se o abrigo pet correspondente for encontrado, o link correspondente é criado. Do contrário, a url fallback será a url geral do Pets Rs.
[x] O link deve ser claramente identificado e fácil de clicar.
[x] O link deve abrir a página do abrigo de pets em uma nova aba.
[x] A URL do link deve ser gerada dinamicamente com base nos dados do abrigo de pet.
[x] Garantir que o link esteja disponível apenas para abrigos de pets.

EVIDÊNCIAS DA ALTERAÇÃO

rhuam commented 4 months ago

@coelhotatiane acho que você pode simplificar bastante sua lógica se incluir uma coluna na tabela shelters chamada, por exemplo, "petsrs_id", se preenchida, ela exibe o link para o site pets rs. O que acha?

coelhotatiane commented 4 months ago

@rhuam então, acho que simplificaria no front sim, mas o back ainda teria que ter essa lógica + algum job recorrente pra adicionar e atualizar os dados no banco. Além disso, os dados dessa forma poderiam não estar tão atualizados quanto estão agora (fazendo a chamada diretamente).

Pela simplicidade optei por deixar no front.. mas se preferir trocar me avisa

coelhotatiane commented 3 months ago

parabéns pelo trabalho @coelhotatiane 🚀

rodei seu código localmente e está funcionando belezinha, mas acho que podemos melhorar um pouco a estilização se seguirmos a sugestão da @barbiebrega aqui pra deixar o link mais evidente, o que acha?

ps: estou avaliando primeiro a funcionalidade, após o ajuste avaliarei o código 🤝

Done!

SOS-RS / frontend

feat: add PetsRs link to shelter card #302