Site do BackEndBrasil - Form de pesquisa de Vagas
onhernandes opened this issue · 6 comments
Oi!
No site do BackEndBrasil estamos com um pequeno bug na área de vagas; quando o usuário aplica um filtro e logo carrega mais issues do Github, o filtro não é aplicado para as issues vindas da API.
Abriram uma issue aqui relatando esse problema.
Hoje, o filtro todo é bem simples e feito todo no lado do navegador, devido algumas implicações com a API do Github. Ainda vamos trabalhar em uma API nossa para melhorar isso, mas até lá, vamos manter no navegador mesmo.
Se alguém puder e quiser ajudar, agradecemos.
O filtro é feito em JS puro mesmo e temos Babel pra transpilar o código todo. Se quiserem refatorar, fiquem a vontade.
Skills necessárias
Apenas JS =)
O repositório do site é esse aqui
Valeu!
seria esse o problema?
comentei la na issue, com uma possivel solução que envolve pouco código, e que soluciona a seleção dos estados no filtro!
o problema de não carregar quando clicado, é somente CSS, vou abrir uma PR!
@rafa-acioly não exatamente, mas pode implicar no filtro.
O filtro é aplicado no navegador, e quando carrega mais issues, o filtro não é aplicado para novas issues. Então tu pode filtrar apenas por SP e quando carregar mais issues, vir algumas de Minas Gerais e exibir ambas, porque o filtro não foi aplicado novamente.
@onhernandes Estou vendo isso agora, na verdade ele aplica, porém aplica também um filtro pra uma tag "language" que dá 'undefined' que parece que não existe, acabei de só ignorar ela e tudo corre normalmente (um simples if)
Mas esse problema escondia outros, como as vagas se tornam infinitas porque o botão carregar mais não funciona como deveria (por não ter um filtro por trás), ele carrega uma pagina que pode não ter nenhuma vaga do estado, e não carregar nada, e quando chega ao fim, repete a ultima vaga infinitamente enquanto o usuário clicar no botão.
@Pliavi isso é uma coisa que quero resolver com uma API futura. Mas não vai ser muito cedo hahaha to organizando algumas coisas e a resolução disso seria só paliativa, no caso dessa tag languague. Pode mandar um PR lá com esse if mesmo? Só pra servir de correção paliativa até a gente ter uma API nossa.
Problema resolvido, obrigado todo mundo =)