search

Cv-Keep / cvkeep-backend

Cv Keep back end official repository
https://cvkeep.com
73 stars 5 forks source link

Url de background #6

Closed LuccaPrado closed 2 years ago

LuccaPrado commented 2 years ago

Percebi que nas chamadas é possível alterar a url da foto para uma que não esteja na lista, exemplo:

image

Esse bug parece perfeitamente manipulável para virar uma feature rsrs

LuccaPrado commented 2 years ago

Abri como back pois é pela tratativa do back, mas, fiquei em dúvida se deveria ser front

felippe-regazio commented 2 years ago

Cara, vc acha que pode ser util transformar esse bug em feature e deixar o usuário setar uma URL custom pra background?

LuccaPrado commented 2 years ago

Acredito que sim, mas, colocando um lembrete que né, o cv é a forma como a empresa o vê, portanto n pode avacalhar. Ou até só permitir de sites específicos como pexels ou outros bancos de fotos que não tem mta coisa indecente/para avacalhar

rntvicente commented 2 years ago

@LuccaPrado a questão é a uri da imagem de foto que não tem um tratamento para aceitar as extensões de imagem ou aquele nome que está Dev "" <S>, não consegui entender bem, pode me ajudar ?

rntvicente commented 2 years ago

@felippe-regazio Pode ajudar esclarecendo qual real feature que deve ser implementada, acabei ficando confuso com a issue?

felippe-regazio commented 2 years ago

@rntvicente a questão do @LuccaPrado era que é possível forçar o uso de uma URL que é diferente das imagens disponibilizadas pelo sistema como imagens de background. TIpo, o sistema tem uma lista pre determinada que vc pode usar, mas vc consegue fazer um override da url original por qualquer outra url.

felippe-regazio commented 2 years ago

Bom, implementei uma validação do path usando no background, e vou deixar como feature no roadmap integração com Unsplash. Estou fechando a issue como solved. Obrigado todo mundo que contribuiu com o report e com o debate.