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

SOFT-3734 #9

Merged
merged 5 commits into from
Jul 26, 2024
Merged

SOFT-3734 #9

merged 5 commits into from
Jul 26, 2024

Conversation

ninedev-i
Copy link
Contributor

Добавил страницу Статьи, которая сама генерируется из списка статей
image
@aadegtyarev @lostpoint-ru @webconn @KraPete

@ninedev-i ninedev-i requested a review from aadegtyarev July 24, 2024 12:56
@aadegtyarev
Copy link
Contributor

aadegtyarev commented Jul 24, 2024

Вот тут то, о чём я говорил. Нужна возможность вместо url написать в файле статью. Видишь ссылка на наш сайт?
изображение

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

Ещё мне не нравится формат даты и вообще, что дату надо писать. Может совсем выкинем дату?

@ninedev-i
Copy link
Contributor Author

ninedev-i commented Jul 24, 2024

Вот тут то, о чём я говорил. Нужна возможность вместо url написать в файле статью. Видишь ссылка на наш сайт?

Это разные задачи, думаю при реализации задачи с отдельными внутренними страницами статей там будет вместо полноценной ссылки -- относительный путь. Впрочем к текущей доброске это не имеет никакого отношения

Ещё мне не нравится формат даты и вообще, что дату надо писать. Может совсем выкинем дату?

Формат даты это база ISO 8601. А как сортировать если не по датам? Плюс пользователю может быть интересно почитать более свежие статьи.

@aadegtyarev
Copy link
Contributor

aadegtyarev commented Jul 25, 2024

Это разные задачи, думаю при реализации задачи с отдельными внутренними страницами статей там будет вместо полноценной ссылки -- относительный путь. Впрочем к текущей доброске это не имеет никакого отношения

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

Что хочу я, как заказчик:

  1. Создавать страницы со статьями и из них формировать автоматом страницу со списком статей.
  2. Если у меня статья снаружи, то я просто не пишу её в теле файла, а даю ссылку в url и вместо открытия страницы внутри сайта открываем внешний url.

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

@ninedev-i
Copy link
Contributor Author

Это разные задачи, думаю при реализации задачи с отдельными внутренними страницами статей там будет вместо полноценной ссылки -- относительный путь. Впрочем к текущей доброске это не имеет никакого отношения

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

Что хочу я, как заказчик:

  1. Создавать страницы со статьями и из них формировать автоматом страницу со списком статей.
  2. Если у меня статья снаружи, то я просто не пишу её в теле файла, а даю ссылку в url и вместо открытия страницы внутри сайта открываем внешний url.

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

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

image

@aadegtyarev
Copy link
Contributor

aadegtyarev commented Jul 25, 2024

просто это другая задача.

Ок, не возражаю, чтобы это делалось потом. Просто хочу, чтобы оно было в одном месте и статьи с ссылками и статьи без ссылок.

А зачем в твоём представлении url? Какую роль он играет?

Ещё я подумал, что не знаю, как задать такой странице место на сайте, откуда она будет доступна. Этот как раз этот url?

@ninedev-i
Copy link
Contributor Author

Ок, не возражаю, чтобы это делалось потом. Просто хочу, чтобы оно было в одном месте и статьи с ссылками и статьи без ссылок.

так и будет

А зачем в твоём представлении url? Какую роль он играет?

Сейчас это ссылка на habr, sputhub и внутренние страницы.

Ещё я подумал, что не знаю, как задать такой странице место на сайте, откуда она будет доступна. Этот как раз этот url?

Тут немного сложнее. На данном этапе нам нужно будет дружить основной сайт с этим урлом. Вероятно в дальнейшем если мы захотим полностью перевести сайт сюда, то это будет делать проще - складыванием страниц в определенную папочку.

@aadegtyarev
Copy link
Contributor

aadegtyarev commented Jul 25, 2024

Тут немного сложнее. На данном этапе нам нужно будет дружить основной сайт с этим урлом. Вероятно в дальнейшем если мы захотим полностью перевести сайт сюда, то это будет делать проще - складыванием страниц в определенную папочку.

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

Текущий URL статей на сайте мы не сможем менять, так как они висят в поисковиках. Но мы можем тупо ставить редирект со старых URL на новые. Поэтому при создании страницы в гите новым инструментом мы можем просто ставить редирект на сайте, как это обычно делаем.

p.s. случайно закрыл RP

@aadegtyarev aadegtyarev reopened this Jul 25, 2024
Copy link
Contributor

@aadegtyarev aadegtyarev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом по этому этапу ок, аппрув. Про редиректы написал в комментах, это может сильно облегчить нам жизнь при переезде на новый движок.

@ninedev-i ninedev-i merged commit 0f093ae into main Jul 26, 2024
1 check passed
@ninedev-i ninedev-i deleted the feature/articles branch July 26, 2024 08:53
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.

2 participants