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

docs: describe local browsers #49

Merged
merged 1 commit into from
Dec 24, 2024
Merged

Conversation

KuznetsovRoman
Copy link
Member

No description provided.

Copy link

github-actions bot commented Dec 23, 2024

✅ Successfully deployed static

Copy link
Member

@shadowusr shadowusr left a comment

Choose a reason for hiding this comment

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

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

@@ -9,7 +9,7 @@ import Admonition from "@theme/Admonition";
import runTestsFromSidebarVideoUrl from "/video/blog/vscode-extension/run-tests-from-sidebar.mp4";
import runTestsFromEditorVideoUrl from "/video/blog/vscode-extension/run-tests-from-editor.mp4";

Для Testplane реализовали расширение для [VS Code][vscode], с помощью которого можно настраивать Testplane с нуля, запускать тесты и удобно работать с <a href="/ru/docs/v8/command-line/#testplane-repl">REPL режимом</a>.
Для Testplane реализовали расширение для [VS Code][vscode], с помощью которого можно настраивать Testplane с нуля, запускать тесты и удобно работать с <a href="/ru/docs/v8/command-line#testplane-repl">REPL режимом</a>.
Copy link
Member

Choose a reason for hiding this comment

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

А что не так со слешами? У нас все страницы на сайте оканчиваются на /. Вроде бы есть редирект с page на page/, но зачем?

Также давай пофиксим битую ссылку, раз ты трогаешь этот файл

Screenshot 2024-12-23 at 18 27 17

Copy link
Member Author

@KuznetsovRoman KuznetsovRoman Dec 23, 2024

Choose a reason for hiding this comment

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

А что не так со слешами?

Дважды менял эту ссылку: сначала с command-line/ на command-line/main, потому что реализовал install-deps как подстраницу command-line, потом вставил install-deps в command-line и тут изменил уже обратно, с command-line/main на command-line

давай пофиксим битую ссылку

На самой странице статьи все работает корректно. Починил в превью, заюзав html вместо md

blog/local-browsers.mdx Outdated Show resolved Hide resolved

import Admonition from "@theme/Admonition";

В Testplane добавлена возможность автоматической загрузки браузеров и драйверов для последующего локального запуска.
Copy link
Member

Choose a reason for hiding this comment

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

Предлагаю сюда вставить короткий скринкаст длиной 30-60 секунд (с autoplay + muted), чтобы он автоматом как гифка начинал играть, с демонстрацией новой фичи. Мне кажется это сильно повышает понятность фичи и привлекательность контента в целом.

Copy link
Member Author

Choose a reason for hiding this comment

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

Могу вставить 12 секунд

docs/command-line/index.mdx Outdated Show resolved Hide resolved
docs/config/browsers.mdx Outdated Show resolved Hide resolved
import Admonition from "@theme/Admonition";

# Как запустить Testplane в локальном браузере

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
Member Author

Choose a reason for hiding this comment

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

Я как-то не представляю, куда конкретно его там вставить. Если во "введение", то может быть непонятно, потому что "что происходит" описывается дальше.

В общем, оставил демку только в блоге.

@KuznetsovRoman KuznetsovRoman force-pushed the TESTPLANE-376.local_docs branch from 2564ac5 to 27038e9 Compare December 24, 2024 02:49
@KuznetsovRoman KuznetsovRoman merged commit d39f1c3 into master Dec 24, 2024
2 checks passed
@KuznetsovRoman KuznetsovRoman deleted the TESTPLANE-376.local_docs branch December 24, 2024 22:35
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