[EpicgamesBridge] New bridge #1709
Conversation
There was a problem hiding this comment.
Круто! Но надо исправить.
Notice: Undefined property: stdClass::$image in /var/www/rss-bridge/pr1709/bridges/EpicgamesBridge.php on line 56
Notice: Undefined property: stdClass::$author in /var/www/rss-bridge/pr1709/bridges/EpicgamesBridge.php on line 50
Notice: Undefined property: stdClass::$content in /var/www/rss-bridge/pr1709/bridges/EpicgamesBridge.php on line 52
Такие вещи всплывают, если в корневом каталоге rss-bridge создать пустой файл DEBUG.
|
С названием коммитов можно не париться. Все равно солью в один. |
|
Спасибо @otakuf ! |
|
@em92 Нашел очень жирный косяк. На сайте эпиков, может выйти по два, три и больше постов сразу. Через пару минут, они могут закрепить две поста и они больше не попадают в листинг, что странно. В закрепе могут висеть пару недель, а новости будут дальше обновляться. В итоге в фид закрепленные не попадут. Новости нужно брать с двух источников, судя по их api. На тесте уже новую логику написал. Забираю данные, соединяю, сортирую по дате, вывожу. Тест пока пусть поработает, если всё хорошо, через пару недель новый реквест отправлю. |
Ок. Ждем |
Adds bridge for epicgames.com news page.
Two language selection and count option.