Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Собирает фиды в OPML-файлы и деплоит на сайт #9

Open
wants to merge 37 commits into
base: main
Choose a base branch
from

Conversation

jag-k
Copy link

@jag-k jag-k commented Apr 19, 2021

#1

@pepelsbey
Copy link
Member

Это интересный ход, но мне кажется, что он слишком сложный и нарушает простоту проекта.

Источником правды хочется оставить ридми, так нагляднее. То есть скрипт нужен кастомный, без Eleventy, просто на Node.js. Это стоило сформулировать в ТЗ, но вот формулирую здесь, извини.

@jag-k
Copy link
Author

jag-k commented Apr 20, 2021

Да, хорошо, переделаю)

@jag-k
Copy link
Author

jag-k commented Apr 26, 2021

@pepelsbey добавил немного магии в код, но зато теперь работает))

@timmarinin
Copy link
Collaborator

@jag-k в текущей магии всё ещё нужны десятки индивидуальных файлов? :)

@jag-k
Copy link
Author

jag-k commented Apr 29, 2021

@jag-k в текущей магии всё ещё нужны десятки индивидуальных файлов? :)

Уже не нужны, выпилю из пула сейчас их))

@pepelsbey
Copy link
Member

Пока починил зависимости и сборку, смотрю дальше )

@jag-k
Copy link
Author

jag-k commented Apr 30, 2021

Скорее всего забыл добавить в package.json зависимость)

@jag-k
Copy link
Author

jag-k commented May 3, 2021

Добавил создание разных файлов (для русского и английского), а так же "плоскую" версию. Некоторые OPML читалки не принимают вложенные outline элементы (такие как RSS Mobile для iOS)

@jag-k
Copy link
Author

jag-k commented May 16, 2021

@pepelsbey, Обновил README.md до актуального состояния, а так же обновил все OPML-файлы

@jag-k
Copy link
Author

jag-k commented May 31, 2021

@marinintim @pepelsbey хотелось бы узнать, что с этим PR?

@pepelsbey pepelsbey changed the title Добавление автоматического создания feeds.opml и обновления README.md Собирает фиды в OPML-файл и деплоит на сайт May 31, 2021
@pepelsbey pepelsbey changed the title Собирает фиды в OPML-файл и деплоит на сайт Собирает фиды в OPML-файлы и деплоит на сайт May 31, 2021
@pepelsbey
Copy link
Member

@jag-k с этим пиаром такая штука, что в нём было много дерзких идей, которые не очень подходили проекту. Я один раз направил, теперь направил ещё раз :) Мне кажется, что в будущем лучше сначала обсуждать, а потом писать код.

Основная идея сейчас такая: в репозитории хранятся фиды и скрипт их сборки в OPML. А сами файлы деплоятся на сайт — ровно так же у нас работает календарь. Тут почти всё готово, надо только доделать:

  • Настроить имена файлов и сократить их количество, см. ридми
  • Настроить сервер, чтобы он редиректил запросы (это я сделаю)

И кажется всё заработает :)

@jag-k
Copy link
Author

jag-k commented May 31, 2021

@jag-k с этим пиаром такая штука, что в нём было много дерзких идей, которые не очень подходили проекту. Я один раз направил, теперь направил ещё раз :) Мне кажется, что в будущем лучше сначала обсуждать, а потом писать код.

Основная идея сейчас такая: в репозитории хранятся фиды и скрипт их сборки в OPML. А сами файлы деплоятся на сайт — ровно так же у нас работает календарь. Тут почти всё готово, надо только доделать:

  • Настроить имена файлов и сократить их количество, см. ридми

  • Настроить сервер, чтобы он редиректил запросы (это я сделаю)

И кажется всё заработает :)

Буквально пару минут и всё будет готово))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants