Plots2: Links externos contendo um símbolo `@` at são atualmente analisados ​​como nomes de usuário de laboratórios públicos.

Criado em 17 mar. 2020  ·  24Comentários  ·  Fonte: publiclab/plots2

Cenário:
Ao inserir um link externo em uma postagem de laboratório público
E o link contém um @
Em seguida, deve ser analisado como parte do URL e não como um link para um nome de usuário de laboratório público

Captura de tela de @joyofsoy :
Screen Shot 2020-03-17 at 3 47 42 PM

Aqui está um link de exemplo para testar a imagem acima: https://medium.com/@erinargyle/working-during-covid-19-how-to-be-good-at-video-meetings-57f49fdb8dcd

Uma solução alternativa atual é usar um site como bit.ly para ofuscar os links, mas o ideal seria corrigir isso. Obrigado @joyofsoy por nos reportar! : tada:

Ruby help wanted ws

Comentários muito úteis

@emilyashley posso trabalhar nisso?

Todos 24 comentários

Notas de Jeff:

@jywarren : Acho que precisamos alterar o comparador de padrão https://github.com/publiclab/plots2/blob/acd5a4910632c8cc80f9fadb54d8cc64c9600519/config/initializers/constants.rb#L2

Aqui está uma demonstração da falha de correspondência de padrão com base nos links originais de Joe; ele deve ser modificado para NÃO reconhecer nomes de usuário dentro de URLs: https://rubular.com/r/HsxxuTG8zG0zSL
Screen Shot 2020-03-17 at 3 57 45 PM

@emilyashley posso trabalhar nisso?

@Tlazypanda isso seria super, obrigada !!!

Olá @Tlazypanda , você teve a chance de experimentar alguma mudança usando este editor de expressão? https://rubular.com/r/HsxxuTG8zG0zSL Parece um joguinho, para tentar ajustar como os ' @names ' são reconhecidos em nosso site, mas não quando há um '/' em ambos os lados.

Ei @ebarry , sinto muito ter esquecido totalmente desse assunto enquanto trabalhava em outros problemas ... está tudo bem se eu retomar depois de alguns dias desde que meus exames na universidade estão acontecendo: sweat_smile:

Claro!!! Boa sorte nos exames !!!

([^`\w]|^)(?<!\/)\@([\w-]+)\b(?!\/)

Parece estar funcionando corretamente. Você pode confirmar se este regex está correto ou não é @ebarry ?

Screenshot from 2020-12-17 23-24-02

@jywarren Você pode confirmar isso?

Gostaria de poder confirmar de qualquer maneira, mas não estou familiarizado com regex, desculpe pela demora na resposta e OBRIGADO por trabalhar nisso!

Funciona perfeitamente! https://rubular.com/r/jfKfgjdqi00qy4

image

Isso agora pode ser inserido em constants.rb - seria um bom problema apenas para iniciantes também!

Isso foi marcado como um bom candidato para se tornar um problema exclusivo para iniciantes como este , o que significa que é simples, independente e com alguma formatação extra, pode ser um ótimo ponto de entrada para um novo contribuidor. Se você estiver familiarizado o suficiente com este código, considere reformatá-lo ou reenviá-lo como um problema exclusivo para iniciantes e , em seguida, execute ping em @ publiclab / revisores para rotulá-lo. Ou, se não for a primeira vez, tente resolver você mesmo!


Obrigado por revisar este @jywarren! Obrigado Gaurav Ahlawat por fazer

esta!

+1 336-269-1539 / @lizbarry http://twitter.com/lizbarry / lizbarry.net

Na terça, 5 de janeiro de 2021 às 4:55 PM Jeffrey Warren [email protected]
escrevi:

Funciona perfeitamente! https://rubular.com/r/jfKfgjdqi00qy4

[imagem: imagem]
https://user-images.githubusercontent.com/24359/103704067-a4875580-4f76-11eb-81ae-51ab06863ecb.png

Isso agora pode ser inserido em constants.rb - seria uma boa
problema exclusivo para os novatos também!

Este foi considerado um bom candidato para se tornar apenas um novato
emitir http://www.firsttimersonly.com como estes
https://code.publiclab.org/#r=all , o que significa que é simples,
independente e com alguma formatação extra, pode ser uma ótima entrada
ponto para um novo contribuidor. Se você estiver familiarizado o suficiente com este código,
considere reformatá-lo ou reenviá-lo como um problema exclusivo para os iniciantes
https://publiclab.org/notes/warren/10-31-2016/create-a-welcoming-first-timers-only-issue-to-invite-new-software-contributors ,
e, em seguida, ping @ publiclab / reviews
https://github.com/orgs/publiclab/teams/reviewers para que seja rotulado.

Ou, se não for a primeira vez, tente resolver você mesmo!

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/publiclab/plots2/issues/7675#issuecomment-754925581 ,
ou cancelar
https://github.com/notifications/unsubscribe-auth/AABHNH47FTWY45FYH34CQCDSYODDNANCNFSM4LN2YLEA
.

@ebarry Você poderia, por favor, convertê-lo em FTO, para que outros possam resolvê-lo? Obrigado!

ótimo trabalho @ gauravahlawat81 Posso fazer isso por você ... ou talvez você gostaria de tentar converter isso sozinho? ... Estou feliz em ajudar onde preso ... o que você acha?

Sim, claro, eu gostaria de convertê-lo sozinho. Como faço isso ?

Devo criar um novo problema ou editar este?

Ótimo: tada :, sim, criar um novo seria mais simples, quando você clicar em um novo problema aqui https://github.com/publiclab/plots2/issues/new/choose selecione First timer Only Issue e ele irá preencher previamente um modelo ... após esta linha onde você tem **Update** the file [$FILENAME]($BRANCH_URL) in the `$REPO` repository (press the little pen Icon) and edit the line as shown below. indique o arquivo que precisa ser alterado e as alterações necessárias ... Dê a ele um título e isso deve ser tudo

Certo, obrigado @cesswairimu , vou tentar!

@cesswairimu Como faço para gerar a diferença?

Não tenho certeza se existe outra maneira de fazer isso ... a forma como eu faço é usando o bot do primeiro temporizador
Então, com essa abordagem, o que você faz é navegar até o arquivo que deseja editar, neste caso, acredito que seja https://github.com/publiclab/plots2/blob/main/config/initializers/constants.rb e clique no botão de edição (ícone de lápis), faça as alterações necessárias e, em seguida, na caixa de texto abaixo deste Create a new branch for this commit and start a pull request edite e certifique-se de que começa com first-timers- e clique em confirmar alterações ...

depois disso, você deve ver seu problema criado aqui https://github.com/publiclab/plots2/issues/ com o autor como bot de primeira hora
mais documentação sobre isso está aqui se isso não fizer sentido

Eu criei o problema, embora tivesse que usar uma abordagem diferente. Por favor, veja se está tudo bem aí e se algo precisa ser mudado, me avise.

O problema é perfeito ... obrigado novamente

Corrigido em https://github.com/publiclab/plots2/pull/8972 Obrigado a todos

Obrigado a todos!

Fluxo de trabalho muito bom

+1 336-269-1539 / @lizbarry http://twitter.com/lizbarry / lizbarry.net

Na sexta-feira, 8 de janeiro de 2021 às 4:59 AM Cess [email protected] escreveu:

Fechado # 7675 https://github.com/publiclab/plots2/issues/7675 .

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/publiclab/plots2/issues/7675#event-4183687540 ou
Cancelar subscrição
https://github.com/notifications/unsubscribe-auth/AABHNH2CH642A3GQR5SCQR3SY3JPFANCNFSM4LN2YLEA
.

Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

first-timers[bot] picture first-timers[bot]  ·  3Comentários

ebarry picture ebarry  ·  3Comentários

grvsachdeva picture grvsachdeva  ·  3Comentários

grvsachdeva picture grvsachdeva  ·  3Comentários

keshavsethi picture keshavsethi  ·  3Comentários