Запрос функции: при публикации в PWA Discourse ссылаться на заголовок, а не на тело сообщения

Внести вклад Функция
1

Было бы очень здорово, если бы ссылки, которые вы публикуете, заполняли бы заголовок, а не (только) тело сообщения, чтобы заголовок заполнялся автоматически, точно так же, как это происходит при публикации внутри сайта.

Есть ли какая-то причина, по которой целью является тело сообщения?

1 лайк
Share to Discourse (via Android)
2

Примете ли вы здесь pull-запрос или есть причина, по которой это было сделано именно так?

@Falco, какой pull-запрос добавил эту функцию (я посмотрел, но не смог найти его)?

3

Это было очень-очень давно, но разве это не сломается, когда вы делитесь не ссылкой, а текстовым содержимым?

По крайней мере, вам нужно будет определить, является ли текст ссылки, и отправить его в соответствующее поле.

1 лайк
4

Да, это, вероятно, сложнее, ведь вы ведь просто делаете GET-запрос к URL, например, https://blah.com/new-topic? ...

Но я с радостью готов посмотреть.

1 лайк
5

Если сейчас большинство сайтов отправляют только URL, я был бы рад таким изменениям.

1 лайк
6

Я нашел код :detective: и проверю, можем ли мы обрабатывать оба варианта… но согласен, что URL-адреса, вероятно, отправляются в 99% случаев, так что это стоит учесть…

1 лайк
7

Отлично, я разобрался, как поддерживать оба варианта:

  • текстовые ссылки будут отправляться в тело сообщения
  • заголовки — в поле заголовка
  • ссылки на ресурсы будут отправляться в поле заголовка (а затем отображаться как полностью раскрытый заголовок, а в теле сообщения — как красивая onebox при инициализации со всеми сопутствующими функциями, включая предпросмотр тем)

На данный момент обновление доступно в рабочем плагине:

Я отправлю это как PR, так как это действительно очевидное решение.

Протестировал это на PWA для Android и Windows (Edge), и всё работает отлично.

1 лайк
Is it possible to customize the "Share to [my-Discourse-mobile-app]" behavior on mobile?
8

Пул-реквест доступен здесь:

9

Напоминаю @Falco

2 лайка
10

Объединено, спасибо за PR!

2 лайка
11

Спасибо за отзыв!

2 лайка
12

Подтверждено: это работает в последней сборке

Ещё одна причина выбрать Android вместо iOS для следующего телефона :mobile_phone:, если вы зависли в мире Discourse :sweat_smile:

Думаю, можно отметить это как выполненное?

3 лайка