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

feat(changelog): add support for changelog blocks #226

Merged
merged 8 commits into from
Jun 1, 2023

Conversation

Feverqwe
Copy link
Contributor

No description provided.

}
}

tokens.splice(openAt, closeAt + 1 - openAt);
Copy link
Member

Choose a reason for hiding this comment

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

Я не уверен, что в RFC имелось в виду такое поведение.
Думаю ожидаемый результат, что ченжлог порендерится в виде обычного md документа и вырежутся конструкции {% changelog %} и {% endchangelog %}

Ты же вырезаешь конструкции вместе с контентом.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

нет, наоборот, эти секции не должны быть видны в доке. Это как бы сниппеты, которые показываются на сайте, а полная дока тут

Copy link
Member

Choose a reason for hiding this comment

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

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

Copy link
Contributor Author

@Feverqwe Feverqwe May 23, 2023

Choose a reason for hiding this comment

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

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

Copy link
Member

Choose a reason for hiding this comment

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

Тогда я не понимаю зачем внутри ченжлога используется md разметка.
Можно все поместить в метаинформацию.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ну можно вставлять ссылки, форматировать текст итп

env.changelogs = [];
}

env.changelogs.push(change);
Copy link
Member

Choose a reason for hiding this comment

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

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ненене, мы складываем json рядом, вот тут diplodoc-platform/cli#348 и там все аккуратно оно харнится

@Feverqwe Feverqwe changed the title Add changelog block feat(changelog): add support for changelog blocks Jun 1, 2023
@Feverqwe Feverqwe force-pushed the rndnm/UXRFC-31/changelogs branch from 9a8108e to 1c3ac59 Compare June 1, 2023 14:35
@Feverqwe Feverqwe merged commit 8f5670b into master Jun 1, 2023
@Feverqwe Feverqwe deleted the rndnm/UXRFC-31/changelogs branch June 1, 2023 14:50
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