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

Netrunners: настройки, аудитории #16

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

Conversation

Tovardas
Copy link

No description provided.

@DeadBread-001
Copy link

Макаров Никита - аппрув

@UserNameShouldBeHere
Copy link

Сапранов Дима - аппрув

- Настройки. Общие. Ошибка "Обязательное поле" при пустом поле "ФИО"
- Настройки. Общие. Ошибка "Обязательное поле" при пустом поле "ИНН"
- Настройки. Общие. Ошибка "Минимальная длина 12" при вводе ИНН короче 12 символов
- Настройки. Общие. Невозможно ввести более 12 символов в поле "ИНН"

Choose a reason for hiding this comment

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

а что будет? ошибка?

Copy link
Author

Choose a reason for hiding this comment

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

Написал подробнее

- Настройки. Общие. Ошибка "Минимальная длина 12" при вводе ИНН короче 12 символов
- Настройки. Общие. Невозможно ввести более 12 символов в поле "ИНН"
- Настройки. Общие. Ошибка "Некорректный ИНН" при вводе в поле "ИНН" не цифры
- Настройки. Общие. Поле "Язык интерфейса". При нажатии на кнопку "RU" в поле "Язык интерфейса" язык интерфейса меняется на русский

Choose a reason for hiding this comment

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

это в футере или в настройках? если в футере, то у вас уже есть эта проверка в команде

Copy link
Author

Choose a reason for hiding this comment

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

Есть и в футере, и в настройках

Copy link

Choose a reason for hiding this comment

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

а, ок тогда

*
![image](https://github.com/user-attachments/assets/4fcccaa6-5db9-4126-a5d6-60a9b269d798)

* Даты. При наведении на кнопки "Сегодня", "Вчера", "Эта неделя", "Прошлая неделя", "Этот месяц", "Прошлый месяц", "Последний 31 день", даты на календаре, попадающие в соответствующий диапазон, меняют свой цвет на более темный.

Choose a reason for hiding this comment

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

давай сделаем кнопки периодов времени или как-то так их назовем, в скобочках можно написать первые 2 варианта, чтобы в автотесте потом не проверять все

Copy link
Author

Choose a reason for hiding this comment

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

Сделано

![image](https://github.com/user-attachments/assets/4fcccaa6-5db9-4126-a5d6-60a9b269d798)

* Даты. При наведении на кнопки "Сегодня", "Вчера", "Эта неделя", "Прошлая неделя", "Этот месяц", "Прошлый месяц", "Последний 31 день", даты на календаре, попадающие в соответствующий диапазон, меняют свой цвет на более темный.
* Даты. Сегдняшняя дата имеет нижнее подчеркивание. Крайние даты диапазона имеют синий цвет, а даты внутри диапазона имеют светло-серый цвет.

Choose a reason for hiding this comment

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

СегОдняшняя

Copy link
Author

Choose a reason for hiding this comment

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

Исправил

- В конце страницы отображается общее количество материалов
![image](https://github.com/user-attachments/assets/5af9fcff-822a-46de-8920-ab38a6d9f82d)

- Пагинатор. Кнопка со стрелкой влево меняет номер текущей страницы на n-1. Если текущая страница первая, то кнопка неактивна.

Choose a reason for hiding this comment

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

может, в отдельный раздел вынести Пагинатор?

Copy link
Author

Choose a reason for hiding this comment

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

Добавил новый раздел "Пагинатор"

@Tovardas Tovardas changed the title Netrunners: настройки, сайты, обзор, обучение, кейсы (без авторизации), обучение (без авторизации), навбар, навбар (без авторизации) Netrunners: настройки, сайты, обзор, обучение, кейсы (без авторизации), обучение (без авторизации), навбар, навбар (без авторизации), Пагинатор Nov 28, 2024
@Tovardas Tovardas changed the title Netrunners: настройки, сайты, обзор, обучение, кейсы (без авторизации), обучение (без авторизации), навбар, навбар (без авторизации), Пагинатор Netrunners: настройки, сайты, обзор, обучение, кейсы (без авторизации), обучение (без авторизации), навбар, навбар (без авторизации), пагинатор Nov 28, 2024
- Настройки. Общие. Ошибка "Минимальная длина 12" при вводе ИНН короче 12 символов
- Настройки. Общие. Невозможно ввести более 12 символов в поле "ИНН"
- Настройки. Общие. Ошибка "Некорректный ИНН" при вводе в поле "ИНН" не цифры
- Настройки. Общие. Поле "Язык интерфейса". При нажатии на кнопку "RU" в поле "Язык интерфейса" язык интерфейса меняется на русский
Copy link

Choose a reason for hiding this comment

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

а, ок тогда

@Tovardas Tovardas changed the title Netrunners: настройки, сайты, обзор, обучение, кейсы (без авторизации), обучение (без авторизации), навбар, навбар (без авторизации), пагинатор Netrunners: настройки, аудитории Dec 8, 2024
https://ads.vk.com/hq/settings

- **Реквизиты.** Изменение поля ФИО и ИНН + нажатие кнопки "Сохранить"
- * Если ФИО "Иван Иванов Иванович" ИНН валидный (312947019840, 756709877700, 482081431745), то измененные данные сохраняются.

Choose a reason for hiding this comment

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

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

![image](https://github.com/user-attachments/assets/259043d7-e483-439e-b1b6-93b1b0189da0)

- * Если в одном из новых полей для Email допущена ошибка, ни одно из новых указанных не добавится.
* Если Email "", то измененные данные **НЕ** сохранаяются, появляется ошибка

Choose a reason for hiding this comment

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

а какая ошибка? может, добавим текст ошибки

- _Ключевые фразы_
- Если заполнить только поле "Ключевые фразы" (Например: учеба, обучение, курсы учебные), то источник и аудитория создадутся успешно
- Если заполнить поля "Ключевые фразы" (Например: учеба, обучение, курсы учебные) и "Минус-фразы" (Например: лень, бездельник), то источник и аудитория создадутся успешно
- Если заполнить только поле "Минус-фразы", то источник и аудитория **НЕ** создадутся

Choose a reason for hiding this comment

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

а будет какое-то предупреждение об ошибке?

- Если выбрать существующую лид-форму и выбрать параметры: "Открытие формы" и "Исключить"; "Отправка формы" и "Исключить", то успешно создастся источник и аудитория
- _События на сайте_
- Если выбрать существующий пиксель и выбрать параметры : "События" -> "Посещение сайта" и "Включить", то успешно создастся источник и аудитория
- Если выбрать существующий пиксель и выбрать параметры : "События" -> "Посещение сайта" и "Исключить", то все другие события станут неактивынми, и успешно создастся источник и аудитория

Choose a reason for hiding this comment

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

неактивными

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.

4 participants