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: Сайты, Мобильные приложения, Настройки, Помощь #8

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

Conversation

Famelbyby
Copy link

No description provided.

@Petr09Mitin
Copy link

Пётр Митин - approved

@Famelbyby
Copy link
Author

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

@endloc
Copy link

endloc commented Nov 18, 2024

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

@glebkos
Copy link

glebkos commented Nov 18, 2024

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

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.

Работа хорошая, можно приступать к дз4
Давай тут добавим скринов, поправим строки про "ошибки" и расставим знаки препинания. Запятых просто нет, из-за этого нужно прям пытаться понять, что описано в кейсе
Спасибо

# Чек-листы

## Сайты

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.

Good day! Fixed)

- Сайты. При добавлении пикселя при вводе некорректного адреса сайта в поле "Домен сайта" и нажатии кнопки "Добавить пиксель" появляется соответствующая ошибка
- Сайты. При добавлении пикселя при вводе некорректного адреса сайта в поле "Домен сайта" и нажатии кнопки "Добавить пиксель" пиксель добавляется в наши пиксели и появляется окно с ID пикселя
- Сайты. При нажатии на кнопку "Получить код" в появившемся окне после добавления пикселя в наши пиксели происходит переход на страницу редактирования пикселя
- Сайты. При добавлении пикселя при вводе некорректного email в поле "Email владельца" и нажатии кнопки "Запросить доступ" появляется соответствующая ошибка

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.

Fixed)

- Сайты. При наведении на пиксель появляется троеточие
- Сайты. При нажатии на троеточие после наведения на пиксель появляется список доступных с пикселем действий
- Сайты. При нажатии на "Переименовать" в появившемся при нажатии на троеточие списке появляется окно с изменением названия пикселя
- Сайты. При вводе символа "№" в поле названия пикселя при переименовании пикселя появляется ошибка

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.

Fixed)

- Сайты. При вводе префикса домена сайта одного или нескольких пикселей в строку поиска показываются только данные пиксели
- Сайты. При вводе не имеющихся префиксов домена сайта или названия пикселей в строку поиска показывается надпись "Ничего не нашлось"
- Сайты. При нажатии на "Настройка" на поле пикселя происходит переход на страницу редактирования пикселя
3. Страница редактирования пикселя. Раздел событий

Choose a reason for hiding this comment

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

Давай, пожалуйста, вставим тут строку, просто для визуального разделения (между 34 и 35, 46 и 47) и т.д.

Copy link
Author

Choose a reason for hiding this comment

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

Fixed)

@markov7785
Copy link

Аппрув)

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