Plots2: Внешние ссылки, содержащие символ `@` at, в настоящее время анализируются как имена пользователей общедоступной лаборатории.

Созданный на 17 мар. 2020  ·  24Комментарии  ·  Источник: publiclab/plots2

Сценарий:
При вставке внешней ссылки в сообщение общедоступной лаборатории
И ссылка содержит @
Затем он должен анализировать как часть URL-адреса, а не ссылку на имя пользователя общедоступной лаборатории.

Скриншот с сайта @joyofsoy :
Screen Shot 2020-03-17 at 3 47 42 PM

Вот пример ссылки для тестирования со скриншота выше: https://medium.com/@erinargyle/working-during-covid-19-how-to-be-good-at-video-meetings-57f49fdb8dcd

Текущий обходной путь - использовать такой сайт, как bit.ly, чтобы скрыть ссылки, но в идеале мы бы это исправили. Спасибо @joyofsoy за сообщение! : тада:

Ruby help wanted ws

Самый полезный комментарий

@emilyashley, я могу поработать над этим?

Все 24 Комментарий

Заметки Джеффа:

@jywarren : Я думаю, нам нужно изменить сопоставление шаблонов https://github.com/publiclab/plots2/blob/acd5a4910632c8cc80f9fadb54d8cc64c9600519/config/initializers/constants.rb#L2

Вот демонстрация сбоя сопоставления с образцом на основе исходных ссылок Джо; его необходимо изменить, чтобы НЕ распознавать имена пользователей внутри URL-адресов: https://rubular.com/r/HsxxuTG8zG0zSL
Screen Shot 2020-03-17 at 3 57 45 PM

@emilyashley, я могу поработать над этим?

@Tlazypanda это было бы супер, спасибо !!!

Привет, @Tlazypanda , у тебя была возможность опробовать какие-либо изменения с помощью этого редактора выражений? https://rubular.com/r/HsxxuTG8zG0zSL Похоже на небольшую игру, чтобы попытаться настроить то, как ' @names ' распознаются на нашем сайте, но не тогда, когда есть '/' с обеих сторон.

Привет, @ebarry, мне очень жаль, что я полностью забыл об этой проблеме, когда работал над другими проблемами ... займусь этим через несколько дней после сдачи моих университетских экзаменов: sweat_smile:

Конечно!!! Удачи на экзаменах !!!

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

Кажется, работает правильно. Не могли бы вы подтвердить, правильно ли это регулярное выражение или нет @ebarry ?

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

@jywarren Вы можете это подтвердить?

Хотел бы я подтвердить в любом случае, но я не знаком с регулярным выражением, извините за мой задержанный ответ и СПАСИБО за работу над этим!

Прекрасно работает! https://rubular.com/r/jfKfgjdqi00qy4

image

Теперь его можно вставить в constants.rb - это тоже будет хорошей проблемой только для новичков!

Это было отмечено как хороший кандидат для став первым старожилы-только вопрос , как это , имея в виду , что это просто, самодостаточным, и с каким - то дополнительным форматированием, может быть большой отправной точкой для нового вкладчика. Если вы достаточно знакомы с этим кодом, рассмотрите возможность его переформатирования или повторной публикации как проблемы , возникающей


Спасибо за просмотр этого @jywarren! Спасибо, Гаурав Ахлават, за то, что

это!

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

Во вторник, 5 января 2021 г., в 16:55 Джеффри Уоррен [email protected]
написал:

Прекрасно работает! https://rubular.com/r/jfKfgjdqi00qy4

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

Теперь его можно вставить в constants.rb - из этого получится хороший
тоже проблема только для новичков!

Это было отмечено как хороший кандидат на то, чтобы стать новичком.
выпускайте http://www.firsttimersonly.com как эти
https://code.publiclab.org/#r=all , что означает, что это просто,
самодостаточный, и с некоторым дополнительным форматированием, может быть отличным входом
балл за нового участника. Если вы достаточно знакомы с этим кодом,
Пожалуйста, рассмотрите возможность переформатирования или повторной публикации, как проблему только для новичков
https://publiclab.org/notes/warren/10-31-2016/create-a-welcoming-first-timers-only-issue-to-invite-new-software-contributors ,
а затем ping @ publiclab / reviewers
https://github.com/orgs/publiclab/teams/reviewers, чтобы пометить его.

Или, если это не ваш первый раз, попробуйте решить проблему самостоятельно!

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/7675#issuecomment-754925581 ,
или отказаться от подписки
https://github.com/notifications/unsubscribe-auth/AABHNH47FTWY45FYH34CQCDSYODDNANCNFSM4LN2YLEA
.

@ebarry Не могли бы вы преобразовать его в FTO, чтобы другие могли его решить? Спасибо!

отличная работа @ gauravahlawat81 Я могу сделать это для вас ... или, может быть, вы хотели бы попробовать преобразовать это самостоятельно? ... Я рад помочь, где застрял ... что вы думаете?

Да, конечно, я бы хотел переделать его сам. Как мне это сделать ?

Должен ли я создать новый выпуск или отредактировать этот?

Отлично: tada:, да, создать новый будет проще, когда вы нажмете на новую проблему здесь https://github.com/publiclab/plots2/issues/new/choose выберите First timer Only Issue и он будет предварительно заполнен шаблон ... после этой строки, где у вас есть **Update** the file [$FILENAME]($BRANCH_URL) in the `$REPO` repository (press the little pen Icon) and edit the line as shown below. укажите файл, который необходимо изменить, и необходимые изменения ... Дайте ему заголовок, и это должно быть все

Хорошо, спасибо @cesswairimu , я попробую!

@cesswairimu Как мне сгенерировать

Не уверен, есть ли другой способ сделать это ... как я это делаю, - это бот-первопроходец.
Таким образом, при таком подходе вы переходите к файлу, который хотите отредактировать, в этом случае я считаю, что это https://github.com/publiclab/plots2/blob/main/config/initializers/constants.rb и нажмите на кнопку редактирования (значок карандаша), внесите необходимые изменения, затем текстовое поле под этим Create a new branch for this commit and start a pull request отредактируйте его и убедитесь, что оно начинается с first-timers- затем нажмите «Принять изменения» ...

после этого вы должны увидеть свою проблему, созданную здесь https://github.com/publiclab/plots2/issues/ с автором в качестве первого таймера-бота
дополнительная документация по этому поводу здесь, если это не имеет смысла

Я создал проблему, хотя мне пришлось использовать другой подход. Пожалуйста, посмотрите, все ли там в порядке, и если что-то нужно изменить, дайте мне знать.

Вопрос идеальный .. еще раз спасибо

Исправлено в https://github.com/publiclab/plots2/pull/8972 Всем спасибо

Спасибо вам всем!

Очень аккуратный рабочий процесс

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

В пятницу, 8 января 2021 г., в 4:59, Cess [email protected] пишет:

Закрыт № 7675 https://github.com/publiclab/plots2/issues/7675 .

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/7675#event-4183687540 или
отказаться от подписки
https://github.com/notifications/unsubscribe-auth/AABHNH2CH642A3GQR5SCQR3SY3JPFANCNFSM4LN2YLEA
.

Была ли эта страница полезной?
0 / 5 - 0 рейтинги