[Telegram] Notifications 2.0.0

[hurtman]

чёта не понял как там изменить, формат немного другой....

Обновил на 2ю бету - та-же самая фигня, как до этого была - двойной /forum/forum/ в уведомлениях :)

p.s. обновился из-за того, что на новой 2.1.6 полезли какие-то ошибки связанные с фильтрацией dom адреса?!

 

[CrazyHackGUT]

двойной /forum/forum/ в уведомлениях

?
Пора тестить в подпапках и изобретать какой-то костыль, если реально такой прикол есть.
А в сообщениях переписки ссылки нормальные?

 

[hurtman]

Хе, о переписке не приходит ничего. При этом если поставить галку - принимать, то она есть до первого уведомления (оно через телегу не приходит), а галка после этого в настройке пропадает.

 

[CrazyHackGUT]

При этом если поставить галку - принимать, то она есть до первого уведомления (оно через телегу не приходит), а галка после этого в настройке пропадает.

Значит Telegram API вернул ошибку. В случае любой ошибки (пока так), оно отписывает. По задумке, должно отписывать (деактивировать получение уведомлений) в случае если пользователь заблокировал сообщения от бота.

 

[hurtman]

Значит Telegram API вернул ошибку. В случае любой ошибки (пока так), оно отписывает.

нет, смотри - уведомления по форуму - приходят. уведомления переписки - нет. снимается галка!

шлёт-то оно одним макаром, но остальное работает, а переписки - нет.

тест из админки, привяка аккаунта и уведомления в разделах - работают. URL в уведомлениях разделов - /forum/forum/

 

[CrazyHackGUT]

Если сейчас соберу тестовый релиз с исправлением дублирования папки - сможете проверить?

 

[hurtman]

с исправлением дублирования папки - сможете проверить?

конечно 8) ещё-бы понять почему личка не доходит.

 

[CrazyHackGUT]

ещё-бы понять почему личка не доходит.

Её сейчас тестанул после своего "теоретического фикса" - пришла. Значит мой косяк был.

 

[hurtman]

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

ага, вот тема с DOM

[LIST]
[*]ErrorException: [E_WARNING] DOMDocument::loadHTML(): error parsing attribute name in Entity, line: 1
[*]src/addons/SModders/TelegramNotifications/Service/HtmlPurifier.php:102
[/LIST]

вылезает на при тыканье лайков. появилась сегодня с 2.1.6, было как на 1 так и на 2 версии вашего плага.

 

[CrazyHackGUT]

появилась сегодня с 2.1.6

*матерится*
Пойду обновлять прод и проверять.

 

[hurtman]

да, ссылки на темы теперь нормальные. остался только косяк с реакцией на лайки, возможно ещё что-то вылезет 8)

 

[CrazyHackGUT]

косяк с реакцией на лайки

Похоже на косяк в движке.
Движок возвращает немного невалидный HTML.


От этого аддон и ломается.
Если сейчас удастся воспроизвести на офе - будем писать баг-репорт.

 

[hurtman]

{posterparams} не должно я так понимаю быть в хтмле страницы?

 

[CrazyHackGUT]

Именно.
Удалось воспроизвести на офе (спасибо West14 за помощь),

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

.

Переместили в дубликаты, ибо уже успели отрепортить. Здорово.
Самое смешное, что в другом репорте ссылаются на другой баг-фикс, который и "спровоцировал" баг.
Пофиксили одно, сломали другое ?

Если в течении недели никакого фастпатча не будет - придётся внедрять костыль против 2.1.6.

 

[CrazyHackGUT]

CrazyHackGUT обновил(а) ресурс [Telegram] Notifications новой записью:

Обновление до 2.0.0 Beta 4

• Исправлена ошибка, когда оповещения из переписок не отправлялись в Telegram.
• Исправлена ошибка, когда при нахождении форума в подпапке, неправильно формировался URL на уведомление.

Спасибо hurtman за репорты и тесты.

Узнать больше об этом обновлении...

 

[Mirovinger]

Именно.
Удалось воспроизвести на офе (спасибо West14 за помощь),

У Вас недостаточно прав для просмотра ссылок. Вход или Регистрация

.

Переместили в дубликаты, ибо уже успели отрепортить. Здорово.
Самое смешное, что в другом репорте ссылаются на другой баг-фикс, который и "спровоцировал" баг.
Пофиксили одно, сломали другое ?

Если в течении недели никакого фастпатча не будет - придётся внедрять костыль против 2.1.6.

А его скорее всего и не будет, всё критичное на их взгляд, они выкатывают сразу.

 

[Matew]

Похоже на косяк в движке.
Движок возвращает немного невалидный HTML.
Посмотреть вложение 110576
Посмотреть вложение 110577
От этого аддон и ломается.
Если сейчас удастся воспроизвести на офе - будем писать баг-репорт.

А зачем ты вообще парсишь готовый html и опять пересобираешь? Создал бы красивые шаблоны с фразами

 

[CrazyHackGUT]

А зачем ты вообще парсишь готовый html и опять пересобираешь?

Не пересобираю, а чищу от атрибутов и лишних HTML-тегов, который Telegram не любит.
Чтобы иметь возможность поддерживать всевозможные виды алертов, включая алерты от аддонов, вроде Article Management System.

 

[hurtman]

a нельзя цепляться к системе уведомлений? тот-же пуш функционал он-же не парсит ничего? ну, я хз как сделано, просто мысли :)

 

[Captain]

Вышел Patch, но там другое поправили)