LINUX.ORG.RU

Разрабатываю LOR API

 

Отправил пуллреквест на вывод топиков по разделу и т.п.

mcgeek ()

По последней ссылке (получение списка тем) обратил внимание на

      "weekyear" : 2007,
      "yearOfEra" : 2007,

это так и задумано? или в weakyear не то возвращается?

bvn13 ★★★★★ ()
Ответ на: комментарий от CYB3R

Очень надеюсь, что он получился не слишком отвратный.

mcgeek ()
Ответ на: комментарий от bvn13

Именно для вывода списка топиков использую встроенные в движок функциии

topicListService.getAllTopicsFeed(section, Date.valueOf(fromDate));
topicListService.getTopics(topicListDto)
Без понятия, что там происходит внутри, еще не смотрел. Вообще, не понятна логика вывода столь большого количества ненужных данных.

mcgeek ()
Ответ на: комментарий от mcgeek

Может, кто-нибудь подскажет, почему здесь отдается кроме топика JSONObject effectiveDate?

mcgeek ()

Было бы неплохо, если maxcom согласится, вынести все URL'ы API в одно место, вроде /api/REQUEST. Сейчас они как только не называются...

mcgeek ()

Код я еще посмотрю. Нужно описание API, давай его в каталоге docs в markdown положим. Потом надо будет все pull request'ы снабжать изменениями в docs чтобы было проще делать review.

maxcom ★★★★★ ()
Ответ на: комментарий от maxcom

Спасибо за одобрение, набросаю тогда, в docs пуллреквест отправлю.

mcgeek ()

За такое представление даты - УБЕВАТЬ!!!! GMT Unixtime более чем достаточно. За атрибут link в ответе - вешать за яйца в жестокой форме. В крайнем случае, возвращай { «reference»: { «section»: NN; «thread»: MM } }, а за твоё { «link»: «/some/bullshit» } надо лупить монтировкой.

no-dashi ★★★★★ ()
Ответ на: комментарий от maxcom

Отправил пуллреквест с документацией (включил созданные методы). Надо будет привести к однообразному виду внутренние методы движка. Что отдавать, как отдавать, имена атрибутов. Иногда отдаваемая информация избыточна. Не подскажете насчет этого?

mcgeek ()
Ответ на: комментарий от mcgeek

Может, кто-нибудь подскажет, почему здесь отдается кроме топика JSONObject effectiveDate?

Не понял вопроса.

maxcom ★★★★★ ()
Ответ на: комментарий от no-dashi

По хорошему не накладно отдавать оба варианта. Первый будет удобнее для обработки, второй - согласно HATEOAS (если абсолютный, с доменом)

vertexua ★★★☆☆ ()
Ответ на: комментарий от mcgeek

Ужас-то какой! Зачем такая тонна ненужной информации?

Eddy_Em ☆☆☆☆☆ ()
Ответ на: комментарий от Eddy_Em

Это движок отдает, самому не нравится. Надо будет поправить.

mcgeek ()
Ответ на: комментарий от mcgeek

Дык, разве ты не движок патчишь?

Там кучу лишнего надо выкинуть и добавить недостающие вещи, чтобы можно было сделать динамический трекер и "бесконечную ленту" в темах. Ну и древообразный вид тем можно будет проще с такой штукой сделать, либо выборку сообщений по определенному критерию (скажем, все ответы на данный комментарий).

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

Eddy_Em ☆☆☆☆☆ ()
Ответ на: комментарий от Eddy_Em

Дык, разве ты не движок патчишь?

Нет, он запиливает функционал.

чтобы можно было сделать динамический трекер и «бесконечную ленту» в темах

Вот он делает лор-API, и пусть делает лор-API.

P.S: А то получится как всегда, и лор-API не будет и «бесконечная лента» тоже пройдёт мимо.

shrub ★★★★★ ()
Последнее исправление: shrub (всего исправлений: 1)
Ответ на: комментарий от Eddy_Em

Надо сначала API стандартизировать, а потом уже можно и движок патчить.

mcgeek ()

Пуллреквест на получение информации о топике влит в master.

mcgeek ()

Пуллреквест на получение последних сообщений трекера влит в master.

mcgeek ()

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

Deleted ()
Ответ на: комментарий от Deleted

Получение топиков уже работает, остальное - когда maxcom синхронизирует ЛОР с репозиторием.

mcgeek ()
Ответ на: комментарий от mcgeek

maxcom, синхронизируй, пожалуйста, ЛОР с гитхабом. Надо работу трекера на Android оттестировать.

mcgeek ()
Ответ на: комментарий от vostrik

По-хорошему они должны отдаваться сразу в списке. Попробую переписать встроенные методы.

mcgeek ()
Ответ на: комментарий от mcgeek

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

vostrik ★★★☆ ()

Чтобы не создавать новую тему до релиза, спрошу здесь. Несколько вопросов по Android приложению:

1. Нужно ли переключение между проверенными новостями/темами галереи и неподтвержденными?

2. Нужен ли просмотр архива и добавление новостей из приложения?

3. Какие цветовые схемы вы бы хотели видеть? (Пока что есть дефолтная, как на ЛОРе, но в настройках сайта еще белая и т.п.). Или же сделать экран для выбора цветов и подгрузки конфигов тем в приложение?

4. По отображению комментариев. Из JSON вполне можно собрать древовидную структуру, какие должны быть отступы между уровнями? Можно сделать точками, как в приложении хабры. Или же как-то по-другому (пишите).

5. Минимальная поддерживаемая версия Android? (Не думаю, что все здесь сидят на последних сборках CM).

Также приветствуются фичреквесты и предложения по интерфейсу. Благодарю за внимание.

mcgeek ()
Ответ на: комментарий от mcgeek

1. Мне нет, но вообще нужно.
2. Добавление комментариев — да, новостей — сомнительно, что оформлять новость удобно на телефоне (на планшете еще куда ни шло, но для него нужно отдельный UI писать). Можно оставить «на потом» для реализации в будущих апдейтах.
3. Желательно иметь все основные темы: «tango», «black», «white2». Но лучше сразу сделать подгрузку конфигов тем. Тот же Эдик тебя замучает, если не сможет оформить просмотр лорчика в вырвиглазных тонах, как в своем юзерскрипте. А понелюбы выберут тебя в президенты ЛОР, если смогут ставить поняш на бэкграунд — мелочь, а приятно.
4. Хоть каком кверху, просто сделай эти долбанные древовидные комментарии :).
5. «4.0». Не нужно поддерживать тостеры с джинджербредом, айскрим сэндвич и > — наше всё.

Также приветствуются фичреквесты и предложения по интерфейсу

Для упоротых нужно обязательно ведение истории (ее лучше архивировать/сжимать для экономии места и ресурса флешки) их «нетленок». По интерфейсу интересного не придумаешь, разве что «пасхалок» спрятать для забавы.

Virtuos86 ★★★★★ ()
Последнее исправление: Virtuos86 (всего исправлений: 1)
Ответ на: комментарий от Virtuos86

5. «4.0». Не нужно поддерживать тостеры с джинджербредом, айскрим сэндвич и > — наше всё.

А может глянуть статистику, кого больше ходит на сайт - с микроволновки или с утюга?

shrub ★★★★★ ()
Ответ на: комментарий от shrub

Найдете такую статистику, обязательно посмотрите, если есть такое желание!

На данный момент мне сложно представить стимул использовать технически устаревшие устройства, а с андроидом второй версии других быть и не может. Маленький экран и всё такое, зачем? Третья версия была ориентирована под планшеты и попросту промежуточной. Имхо, конечно.

Virtuos86 ★★★★★ ()
Ответ на: комментарий от Virtuos86

А в задницу такие имхи. (c)

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

vostrik ★★★☆ ()
Ответ на: А в задницу такие имхи. (c) от vostrik

Всего 5.6%. Нормальные приложения все перешли на API 15+. Исходники в любом случае будут на гитхабе, там и патчики сидящие с микроволновок себе напишут. Но, если будет возможность через support libraries сделать поддкржку и для этих устройств без усложнения кода и костылей, то она будет.

mcgeek ()
Ответ на: комментарий от mcgeek

Всего 5.6%

Это общая по больнице или тебе maxcom дал статистику по ЛОРу?

shrub ★★★★★ ()

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

Алсо, начинаю клиент для айфончиков делать.

Deleted ()
Ответ на: комментарий от Deleted

Еще не принятая в репозиторий документация. Оттуда работают получение трекера, количества уведомлений и их сброс, комментарии, свойства топика.

mcgeek ()

Запиливаю получение профиля пользователя (в т.ч. для работы игнорирования пользователей и тегов в приложении).

mcgeek ()
Ответ на: комментарий от Deleted

Авторизация еще не сделана, помнится, maxcom собирался делать что-то по OAuth.

mcgeek ()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.