Проект

Общее

Профиль

Улучшение #40

Включить смайлики в MOTD

Добавил(а) Павел Рюмин больше 2 лет назад. Обновлено почти 2 года назад.

Статус:
Закрыта
Приоритет:
Нормальный
Назначена:
Александр Трошенков
Дата начала:
04.03.2017
Дата завершения:
Готовность:

50%

Трудозатраты:
Браузер:
Ссылка на картинку:

Описание

https://puu.sh/utHru.png

кейс такой:
Настройки > Редактируем MOTD > Закрываем настройки через крестик > MOTD должен стразу обновляться в чате

Это раньше работало, надо починить

issue40 (347 КБ) Александр Трошенков, 07.03.2017 19:58

История

#1 Обновлено Павел Рюмин больше 2 лет назад

  • Параметр Ссылка на картинку изменился на https://puu.sh/utHru.png

#2 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Новая на В работе

#3 Обновлено Александр Трошенков больше 2 лет назад

Нужно ли проверять смайлики на доступность через сервис SmilesService ?
Если нет, то задача готова

#4 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с В работе на Обратная связь
  • Параметр Готовность изменился с 0 на 50

#5 Обновлено Александр Трошенков больше 2 лет назад

  • Файл issue40 добавлен

#6 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с Обратная связь на В работе

[10:32:40] lokki7 Павел Рюмин: и дублирование кода там
уже есть код, который смайлики вставляет, зачем еще один писать?
[10:32:54] Александр Трошенков: ммм...а я чет его не нашел
[10:35:16] lokki7 Павел Рюмин: public/js/ng/Chat3/ParseMessageDirective.js

вот эту директиву нужно использовать
[10:35:38] Александр Трошенков: Ок, мою директиву выкинуть?
[10:39:29] lokki7 Павел Рюмин: да, смайлики вставляются просто: parse-message="текст со смайликом :peka:"

но в motd еще есть nl2br - нужно не забыть про него
[10:40:27] lokki7 Павел Рюмин: скорее всего заработает просто parse-message="vm.room.motd|nl2br"

#7 Обновлено Александр Трошенков больше 2 лет назад

  • Параметр Статус изменился с В работе на Решена

#8 Обновлено Alexander S около 2 лет назад

  • Параметр Статус изменился с Решена на Новая

Снова не отображаются

#9 Обновлено Александр Трошенков около 2 лет назад

  • Параметр Статус изменился с Новая на В работе

#10 Обновлено Александр Трошенков около 2 лет назад

  • Параметр Статус изменился с В работе на Решена

#11 Обновлено Александр Трошенков около 2 лет назад

  • Параметр Назначена изменился с Верстка на Виктор Федоров

#12 Обновлено Александр Трошенков около 2 лет назад

  • Параметр Статус изменился с Решена на Тестируется

#13 Обновлено Александр Трошенков около 2 лет назад

#14 Обновлено Виктор Федоров около 2 лет назад

  • Описание обновлено (diff)
  • Параметр Статус изменился с Тестируется на В работе
  • Параметр Назначена изменился с Виктор Федоров на Александр Трошенков

#15 Обновлено Александр Трошенков около 2 лет назад

  • Параметр Статус изменился с В работе на Тестируется
  • Параметр Назначена изменился с Александр Трошенков на Тестировщики

Код в ветке wifi-issue40_3

#16 Обновлено Александр Трошенков около 2 лет назад

  • Параметр Статус изменился с Тестируется на В работе

#17 Обновлено Александр Трошенков почти 2 года назад

  • Параметр Статус изменился с В работе на Обратная связь

#18 Обновлено Сергей Охрименко почти 2 года назад

  • Параметр Назначена изменился с Тестировщики на Менеджеры

выглядит рабочим

#19 Обновлено Виктор Федоров почти 2 года назад

  • Параметр Статус изменился с Обратная связь на Закрыта
  • Параметр Назначена изменился с Менеджеры на Александр Трошенков

Экспортировать в Atom PDF