-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Criada ação de copiar DOI na interface responsiva #505
Conversation
…variáveis de ambiente de SOLR_URL E OAI_URL para nossa infraestrutura, o parâmetro -p é interiro que representa a quantidade de horas que iremos coletar os dados do provedor, portanto -p 1 é será hora atual -1 hora
…nefits and legacy of the water crisis in Brazil</field>`
Processamento de coleta de PrePrint
Cria os pipelines: coleção, tipo de documento, autores e títulos
…ine para abrstract
Preprint -> Master
…vedor OAI, mais os teste!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@paratiuid, fiz o teste do PR na branch interface-responsiva
e o botão de cópia do DOI funciona como esperado.
Entretanto, o botão de compartilhamento, por exemplo, parou de funcionar. Entendo que não faz parte deste PR mas, ao incorporar este PR à branch quebrará as implementações anteriores. Poderia verificar?
@patymori Peço que se puder, converse com ele para verificar se é isso mesmo. |
@paratiuid @joffilyfe @robertatakenaka @gustavofonseca Assim como comentado pelo slack, peço que seja dado prioridade para a aprovação do PR: scieloorg/search-journals-proc#4. Esse PR trata de inconsistências na interface, não asseguro que seja esse o motivo, porém a prioridade para esse PR irá evitar comentário e obvervações desnecessárias. |
Video que mostra a validação com artigos em preprint: |
@paratiuid conversando com a equipe ficamos, um pouco preocupados com a futura incorporação do branch Vou explicar esse preocupação já mencionado pela @patymori no comentário: #505 (review) Por enquanto que o branch está seguindo isoladamente da master talvez não enxergamos esse problema, porém é necessário se atentar no momento de incorporação, pois, é possível que as alterações realizadas em javascript quebre outras funcionalidades, isso ocorre quando, por exemplo, é removido um id do HTML e temos um seletor(jquery) esperando por esse id, em fim, além de termos outras alterações sendo realizada pelo @rafaelpezzuto que também irá fundir com a master Para minimizar isso é importante realizarmos antes do envio do PR uma fusão do master com o ramo Dessa forma garantirmos que não teremos quebra de código, ausência de código e ou inconsistências como mencionado no exemplo. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assim como comentado pelo slack, peço que seja dado prioridade para a aprovação do PR: scieloorg/search-journals-proc#4.
Esse PR trata de inconsistências na interface, não asseguro que seja esse o motivo, porém a prioridade para esse PR irá evitar comentário e obvervações desnecessárias.
Revisei os comentários, refiz os testes e realmente funcionou. Peço desculpas pela falta de atenção.
O que esse PR faz?
Adiciona a funcionalidade de copiar DOI na lista de resultados da busca, quando esta está com largura compatível com dispositivos mobile.
Além de copiar o doi, é exibido um feedback deixando o botão verde e exibindo um ícone de check na cor branca, indicando que a ação foi efetuada com sucesso.
Onde a revisão poderia começar?
1 - Acesse a lista de resultados
2 - clique sobre um item da lista
3 - no box de detalhes que aparecerá, localize o botão COPIAR DOI e clique.
Deve ocorrer a animação de feedback e o DOI deve estar na área de transferencia. Tente colar em algum campo de texto para ver se foi copiado corretamente.
Como este poderia ser testado manualmente?
Siga os passo já citados.
Algum cenário de contexto que queira dar?
Ao alinhar sobre esses vários commits com o @jamilatta , entendemos que o processo está correto, pois estamos atualizando a branch interface-responsiva com os ajustes efetuados na master.
Screenshots
Quais são tickets relevantes?
#476
Referências
Um botão semelhante já existe na tela do artigo, no opac.
Foi mantido o mesmo padrão.