GIF-файлы переходят в ядро Discourse!

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

1 лайк

После этого вы всё ещё сможете отключить его через обычные настройки сайта.

@davidb, можете подтвердить?

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

Один из вариантов — сделать это понятнее в заголовке и описании самого изменения, когда оно настраивается.

1 лайк

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

Пока что просто оставьте предстоящее изменение как «Никто», и оно будет отключено по умолчанию в будущем. Нет необходимости вносить какие-либо дальнейшие изменения на стороне администратора.

4 лайка

Да, когда я искал enable_gifs, меня сразу перекинуло на страницу предстоящих изменений. Это было очень запутанно.

Идея раздела «Предстоящие изменения» отличная. Просто нужно немного подправить формулировки.

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

2 лайка

В данном случае @davidb @mcwumbly, если бы тип изменения был site_setting_default, а не feature для enable_gifs, то не появилось бы этого пугающего сообщения о постоянном изменении. Это контролируется здесь:

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

Возможно, добавить will_become_permanent: true или что-то подобное в качестве дополнительных метаданных? Не уверен…

1 лайк

почему нас загнали в использование Klipy вместо других вариантов? Я предпочитаю Giphy и хотел бы продолжать его использовать.

Настройки и взаимодействие с компонентом делают использование других вариантов, таких как Giphy, крайне запутанным.

Думаю, это связано с тем, что Giphy больше не является бесплатным вариантом.

1 лайк

всё в порядке. я разобрался (спасибо, Moin!). мне пришлось удалить компонент discourse gif, чтобы появилось нужное поле настроек. затем я установил значение no one, после чего переустановил и заново настроил компонент. giphy снова работает.

2 лайка

Рада, что смогла помочь. Тот факт, что предстоящее изменение скрыто при установленном компоненте темы GIF, немного запутывает: чтобы использовать компонент, нужно удалить его, отключить предстоящее изменение, а затем переустановить.

Вас уведомляли об изменении, или уведомления были отключены, потому что предстоящее изменение было скрыто? Интересно, приводит ли это к проблеме, когда они не знают, что им нужно запустить миграцию и получить API-ключ.

1 лайк

При подаче заявки на ключ KLIPY требуется указание авторства. Как добавить его во всплывающее окно? И как люди используют его без этого?

1 лайк

О, раньше это было в компоненте темы, но в ядре его нет, там был providerLogo:

Логотип провайдера Klipy был добавлен в FEATURE: Add Klipy as GIF API provider by xfalcox · Pull Request #107 · discourse/discourse-gifs · GitHub . @davidb, было ли намеренным решение не включать этот логотип провайдера в ядро?

2 лайка

К сожалению, похоже, что это было упущено при портировании в основную ветку. Ранее оно отображалось в компоненте темы условно, но теперь мы должны показывать его всегда.

Добавлено здесь: UX: add Klipy branding to gifs picker - Pull Request #41501 - discourse/discourse - GitHub

2 лайка

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

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

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

2 лайка

Не уверен, что требования к указанию авторства вообще были прочитаны. Это всего лишь одно указание авторства, и оно необязательное:


https://docs.klipy.com/attribution

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

ИЗМЕНЕНИЕ: Похоже, что нет, поэтому мы зависим от проверочного видео через Klipy.

привет @fearlessfrog :wave:
у меня была та же проблема — чтобы продолжить использование Giphy, нужно полностью удалить компонент темы (просто отключения недостаточно), затем в настройках admin -> предстоящие изменениявключить GIF выбрать значение “никто”. после этого переустановите форк этого компонента, чтобы использовать его — команда скоро архивирует свою версию. у меня есть форк компонента, который вы можете использовать.

удаление старого компонента:

отключение настройки в admin-предстоящие изменения

я действительно не понимаю, почему команда поступила именно так, но это ужасно запутанно, и мне потребовалось время, чтобы разобраться (спасибо Моину за объяснение) — это очень неудобно для тех, кто размещает серверы самостоятельно. в какой-то момент я, вероятно, просто перестрою форк как свой собственный компонент, убрав другие ненужные поля и файлы, но я добавлю сообщение в баннере администратора, когда это произойдет.

примечание для команды: компонент Discourse Gifs по-прежнему отображается на странице установки компонентов по умолчанию:

4 лайка

Спасибо, @davidb, можешь, пожалуйста, убрать эту опцию?

Мы сделали это именно так, потому что это оптимизирует настройки для наших клиентов, использующих наш хостинг, где мы вручную заходим и удаляем для них старый компонент gif. Мы не предполагали, что люди захотят продолжать использовать Giphy, поскольку их цены были объявлены как очень высокие, что и стало причиной, по которой мы изначально добавили Klipy в тематический компонент.

В следующий раз мы можем добавить немного больше информации в первый пост для тех, кто хостит самостоятельно и хочет продолжать использовать старый компонент, мы просто предположили здесь, что и те, кто хостит самостоятельно, тоже перейдут на Klipy:

Мы обязательно подумаем, как сделать это менее запутанным и неожиданным, я попросил Дэвида Б. создать внутреннюю тему, и я поработаю над этим в ближайшее время.

2 лайка

Не совсем понимаю: я всё ещё использую свой старый бесплатный API Giphy без каких-либо проблем? Я только что вошёл в панель разработчика Giphy, и он там есть. О каких платежах идёт речь, которые я, по слухам, должен нести? Я никогда не вводил данные кредитной карты и не платил ничего, и до сих пор не вижу, где это нужно сделать сейчас — на мой взгляд, всё всегда было бесплатно. Форум, к которому он подключён, не сталкивается с ограничениями по частоте запросов, хотя мы используем его довольно активно :thinking:

Итак, похоже, что всё отлично работает до 100 вызовов API в час, что, я думаю, вполне нормально для тех, кто хостит сервер самостоятельно.

2 лайка

Да, безусловно, но стоимость была бы непомерно высокой для нашего хостинга, учитывая, что мы хотим сделать GIF-анимации основной функцией.

2 лайка

Думаю, в компоненте темы было только «Powered by Klipy» (из этого списка), но при портировании в ядро нам, вероятно, следовало быть более внимательными и сверяться с официальным сайтом — добавил основное обязательное указание в UX: add placeholder text for Klipy gifs search - Pull Request #41651 - discourse/discourse - GitHub

Готово — добавлено в UX: remove gifs from popular theme components - Pull Request #41649 - discourse/discourse - GitHub

Мы обновили логику отображения этого предупреждения. Проблема должна быть решена в DEV: add upcoming change permanent banner metadata - Pull Request #41700 - discourse/discourse - GitHub

Эти проблемы теперь должны быть решены для сайтов, работающих на последней версии Discourse.

4 лайка