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

XRUST_BEZE: Обзор, Кампании, Аудитории, Обучение #7

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

Conversation

Petr09Mitin
Copy link

No description provided.

@Famelbyby
Copy link

Горбунов Алексей - approved

@Petr09Mitin
Copy link
Author

Пётр Митин - approve

@glebkos
Copy link

glebkos commented Nov 17, 2024

Глеб Костин – аппрув ✅

@endloc
Copy link

endloc commented Nov 18, 2024

Амирова Лилиана - approved

Copy link

@markov7785 markov7785 left a comment

Choose a reason for hiding this comment

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

В целом хорошая работа
Попрошу поправить:

  1. Больше картинок го сделаем
  2. Кейсы на редирект переписать
  3. Не ссылаться на https://ads.vk.com/help/articles/user_lists#requirement и похожее, чуть подробнее про валидные и прочее
    Это жду, но думаю, что можете уже начинать 4 дз делать)

Всегда
<ol>
<li>Обзор. Полезные материалы. При нажатом переключателе в положении "Кейсы" в разделе отображаются кейсы.</li>
<li>Обзор. Полезные материалы. При нажатом переключателе в положении "Новости" в разделе отображаются новости.</li>

Choose a reason for hiding this comment

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

Давай добавим фото. Здесь и по возможности как можно подробнее ниже

<li>
При наличии кампаний
<ol>
<li>Обзор. Overview кампаний. При клике по ссылке "Создать кампанию" или по категориям кампаний (Транслируются, Не транслируются, На модерации, Отклоненные) происходит переход на страницу кампаний.</li>

Choose a reason for hiding this comment

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

Кажется это немного разные кейсы, го разобьем

<li>При создании папки с неуникальным названием отображается надпись "Выберите другое имя — это уже занято другой папкой"</li>
<li>При нажатии "Добавить кампании в папку" отображаются кампании и строка поиска по ним</li>
<li>При добавлении кампании в папку она появляется в списке</li>
<li>При добавлении кампании в папку она появляется в списке</li>

Choose a reason for hiding this comment

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

Кажется это дубликат

<ul>
<li>При нажатии кнопки Продолжить и незаполненном бюджете - сообщение об ошибке, перехода не происходит</li>
<li>При нажатии кнопки Продолжить и бюджетом < 100р - сообщение об ошибке, перехода не происходит</li>
<li>При заполненном названии и бюджете >100р нажатие Продолжить переводит пользователя на шаг Группы объявлений создания кампании</li>

Choose a reason for hiding this comment

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

Если так решили расписывать, что уважаю, давайте куда-то включим ровно 100

<li>При нажатии в форме создания каталога на крестик она закрывается</li>
<li>Создание каталога из фида или сообщества. При вводе в инпут ссылки на сообщество текста, не являющегося URL, отображается ошибка "Невалидный url"<li>
<li>Создание каталога из фида или сообщества. При вводе в инпут ссылки на сообщество, в котором нет товаров, отображается ошибка "Укажите ссылку на ваше сообщество ВКонтакте с товарами или услугами"</li>
<li>Создание каталога из фида или сообщества. До ввода в инпут валидного url кнопка "Создать каталог" заблокирована</li>

Choose a reason for hiding this comment

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

Давайте немного опишем и проверим где возможно, что такое валидный
количество символов, язык, символы и т.д.

<li>Лид-формы. Форма создания лид-формы. При непустых полях каждого вопроса и нажатии кнопки "Продолжить" происходит переход к шагу "Результат"</li>
<li>Лид-формы. Форма создания лид-формы. Нельзя ввести пустой заголовок на шаге "Результаты"</li>
<li>Лид-формы. Форма создания лид-формы. При непустом поле "Заголовок" и нажатии кнопки "Продолжить" происходит переход к шагу "Настройки"</li>
<li>Лид-формы. Форма создания лид-формы. На шаге "Настройки" невльзя нажать кнопку "Сохранить" при пустых полях "ФИО" и "Адрес регистрации"</li>

Choose a reason for hiding this comment

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

очепятка

<li>Обзор. Overview кампаний. При клике по ссылке "Создать кампанию" или по категориям кампаний (Транслируются, Не транслируются, На модерации, Отклоненные) происходит переход на страницу кампаний.</li>
<li>Обзор. Overview бюджета. При клике по ссылке "Пополнить бюджет" показывается попап пополнения бюджета.</li>
<li>Обзор. Overview бюджета. Можно менять период, за который отображаются расходы (Сегодня, Вчера, 3 дня и др.)</li>
<li>Обзор. Раздел AdBloger. При нажатии на кнопку "Открыть VK ADBloger" происходит редирект на https://adblogger.vk.com/</li>

Choose a reason for hiding this comment

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

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

Офлайн-конверсии. Создание.
<ul>
<li>Нельзя добавлять файлы форматов, отличающихся от txt и csv</li>
<li>При добавлении файла, не отвечающего требованиям https://ads.vk.com/help/articles/user_lists#requirements, отображаются сообщения об ошибках при парсинге файла</li>

Choose a reason for hiding this comment

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

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

@markov7785
Copy link

markov7785 commented Nov 29, 2024

Аппрув)

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.

5 participants