hack-a-chain-software / indexer-kadena

0 stars 1 forks source link

[Homescreen/Search] Filters are missing from the search box. #71

Open 1Mateus opened 1 month ago

1Mateus commented 1 month ago

The new search option was implemented.

https://github.com/hack-a-chain-software/indexer-kadena/commit/b76c36bd094fc542b2c8c19f7dd23cc85d45dbd7

1Mateus commented 1 month ago

Essa issue vai depender doq vamos ter disponível do endpoint do search: #77

1Mateus commented 4 weeks ago

Fiz no fornt e ja funciona, precisa ainda do backend, mas ja fiz no código e vou deixar rodando

bernardonery commented 3 weeks ago

@davibauer conseguiu dar uma olhada aqui???

rafaeldkb commented 2 weeks ago

@davibauer @1Mateus

Primeiro, um feedback geral pra todos os filtros. Eu achei que ficou meio estranho o nosso "search" agora. Eu não consigo colar um hash ou código ali (seja de um address, transaction, block ou token e simplesmente pesquisar, ele fica ali pesquisando coisas relacionadas. É interessante, mas acho que limitou a funcionalidade de pesquisar direto. Na minha visão, o ideal é que seja possível fazer a pesquisa geral colocando o hash da conta ou da transação e já apertando enter em adição a aparecer ali já filtrado (acho a feature boa, só que apertar "enter" pra tua pesquisa tem que ser possível também).

O filtro de "Transactions" funciona bem, fora o fato de não poder dar o "enter", como comentei acima, mas ele encontra a transação.

A filtro de "address" tá meio confuso. Se eu quero pesquisar um address, ele primeiro aparece as transações daquele address ao invés de me levar pra página do address. E se eu tento mudar de "aba" ali, de transactions pra account, ele não muda, só se eu descer o mouse o address aparece.

O filtro de "blocks" eu também fiquei confuso, quando eu colo um block height ali ele aparece várias transações, mas não a opção de eu clicar e ir pra página de block height. Reforço que deveria ser possível colar o "código" do bloco e ir pra página dele. Mas ao que me parece, várias páginas de bloco tão dando 404. E eu não estou conseguindo ver as transações dos blocos que consegui entrar.

Pro filtro de "tokens", o que eu deveria pesquisar ali? O contrato do token? O nome do token? Eu não consegui entender, nem encontrar a informação do token que eu deveria pesquisar pelo Kadscan (não achei nem o contrato do token, e não consegui pesquisar pelo nome, pois não achou nada).

1Mateus commented 2 weeks ago

@rafaeldkb não consegui entender o seu feedback geral, consegue elucidar?

rafaeldkb commented 2 weeks ago

@1Mateus eu tentei colocar um vídeo, mas não consegui adicionar aqui por algum motivo. A questão é tipo, quando eu colo algum valor ali dentro do campo de search, eu não consigo simplesmente apertar "enter" ou "pesquisar" ali no canto direito. Eu preciso ficar esperando ele buscar os outputs associados a minha pesquisa, pra dai então clicar nele. Eu acho que seria mais legal se eu pudesse escrever/colar um valor ali dentro e apertar "enter/search" e ir direto pra página dele, saca? Como é feito no etherscan e outros. Vou ver se mando um vídeo no slack.

rafaeldkb commented 2 weeks ago

@davibauer tava falando com o Mat e acho que a gente precisa ver isso a nível de backend, sobre como fazer essa query. Vamos ver amanhã na daily, ai eu abro a tela e mostro.

bernardonery commented 2 weeks ago

Testei agora.

Senti um problema na hora da seleção dos filtros durante uma pesquisa. Ele está lento e com problemas na hora da seleção. @1Mateus O que eu fiz foi pegar um endereço e colocar na aba de pesquisa. Ali ele pega o endereço da carteira e transações que essa carteira está relacionada. O padrão é ir para transações, quando você tenta trocar a seleção para Adress, não vai. Eu tive que trocar os filtros para Adress aí sim para ele retornar a carteira correta.

@rafaeldkb como que ficou?

rafaeldkb commented 2 weeks ago

@1Mateus vamo implementar uma lógica no front que nem a gente falou ontem pra quando apertar enter ele ir pro primeiro resultado. No caso da pesquisa de address, que aparece tanto as transações com o address quanto o próprio address, a gente tem que priorizar levar pra um endereço. Então acredito que a lógica vai ser: primeiro checa se é um endereço, ai leva pro endereço, depois pra transação. Deu pra entender?

1Mateus commented 2 weeks ago

@rafaeldkb isso vai depender diretamente doq o backend retornar pro front, vai ter delay, vocês tão ok com isso?

rafaeldkb commented 2 weeks ago

@1Mateus por enquanto, vamos fazer isso, pq se não a gente vai matar a specialized view né. É o tempo de retorno do dado ali. Mas no caso, a lógica do front tem que trazer primeiro o Address e depois a Transação, saca? Tipo, quando eu pesquiso um address, trazer primeiro o address, e não as transações. Ai cria a lógica de quando apertar o enter, vai para a primeira. E isso deve funcionar pra qualquer busca. Se eu pesquisar um block heigh, ele provavelmente vai retornar o de várias chains, ai se eu aperto enter, vai pro primeiro resultado. Deu pra entender? Se precisar, vamos em call.

rafaeldkb commented 2 weeks ago

@davibauer corrigir a query dos tokens para mostrar os polyfungibles também. Avisar aqui quando tiver feito.

1Mateus commented 2 weeks ago

@rafaeldkb Sim, mas o front-end nao sabe o que vc ta pesquisando, eu não consigo inputar prioridade sendo que não vou saber diretamente qual entidade vc quer ver primeiro.

Eu até consigo reorganizar as views do front para o endereço aparecer primeiro, mas isso é estatico e não vai funcionar dinamicamente como vc espera.

Se o backed não retornar transações e nem blocos na view All, a specialized view ja não vai mostrar a lista de transações relacionadas.

A lógica do enter, eu vou fazer como vc pediu, aguardar e usar uma lista de priordade para redirecionar o usuário.

rafaeldkb commented 2 weeks ago

@1Mateus pode ser estático mesmo, pra esse momento não tem problema. Só fazer a lógica do enter mesmo, priorizando address.

1Mateus commented 2 weeks ago

@rafaeldkb fechou

1Mateus commented 2 weeks ago

@rafaeldkb consegue testar o search agora? algum feedback?

rafaeldkb commented 2 weeks ago

@1Mateus ficou show! funcionou bem aqui, valeuuuu, vou dar como done

rafaeldkb commented 2 weeks ago

@davibauer vi que foi implementado os polyfungibles, vou dar como done! Boaaa

1Mateus commented 2 weeks ago

https://github.com/hack-a-chain-software/indexer-kadena/commit/b76c36bd094fc542b2c8c19f7dd23cc85d45dbd7