LINUX.ORG.RU

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

 

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

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

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

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

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

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

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

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

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 ()
Ответ на: комментарий от no-dashi

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

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

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

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

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

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

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

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

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

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

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

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

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

mcgeek ()

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

mcgeek ()

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

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 ()

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

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

Deleted ()

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

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

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

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