-
Notifications
You must be signed in to change notification settings - Fork 29
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
base: main
Are you sure you want to change the base?
Conversation
Пётр Митин - approved |
Горбунов Алексей - approved |
Амирова Лилиана - approved |
Костин Глеб – аппрув ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Работа хорошая, можно приступать к дз4
Давай тут добавим скринов, поправим строки про "ошибки" и расставим знаки препинания. Запятых просто нет, из-за этого нужно прям пытаться понять, что описано в кейсе
Спасибо
# Чек-листы | ||
|
||
## Сайты | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Прям сразу напишу, давай везде добавим скринов, пожалуйста, вообще сложно воспринимать так, спасибо
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good day! Fixed)
XRUST_BEZE-Alexey-Gorbunov.md
Outdated
- Сайты. При добавлении пикселя при вводе некорректного адреса сайта в поле "Домен сайта" и нажатии кнопки "Добавить пиксель" появляется соответствующая ошибка | ||
- Сайты. При добавлении пикселя при вводе некорректного адреса сайта в поле "Домен сайта" и нажатии кнопки "Добавить пиксель" пиксель добавляется в наши пиксели и появляется окно с ID пикселя | ||
- Сайты. При нажатии на кнопку "Получить код" в появившемся окне после добавления пикселя в наши пиксели происходит переход на страницу редактирования пикселя | ||
- Сайты. При добавлении пикселя при вводе некорректного email в поле "Email владельца" и нажатии кнопки "Запросить доступ" появляется соответствующая ошибка |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А давай сразу тут пропишем какая именно, как и везде, где пишем про ошибки
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed)
XRUST_BEZE-Alexey-Gorbunov.md
Outdated
- Сайты. При наведении на пиксель появляется троеточие | ||
- Сайты. При нажатии на троеточие после наведения на пиксель появляется список доступных с пикселем действий | ||
- Сайты. При нажатии на "Переименовать" в появившемся при нажатии на троеточие списке появляется окно с изменением названия пикселя | ||
- Сайты. При вводе символа "№" в поле названия пикселя при переименовании пикселя появляется ошибка |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А это единственный запрещенный символ? Просто если нет и такое пишем, то лучше бы перечислить
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed)
- Сайты. При вводе префикса домена сайта одного или нескольких пикселей в строку поиска показываются только данные пиксели | ||
- Сайты. При вводе не имеющихся префиксов домена сайта или названия пикселей в строку поиска показывается надпись "Ничего не нашлось" | ||
- Сайты. При нажатии на "Настройка" на поле пикселя происходит переход на страницу редактирования пикселя | ||
3. Страница редактирования пикселя. Раздел событий |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Давай, пожалуйста, вставим тут строку, просто для визуального разделения (между 34 и 35, 46 и 47) и т.д.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed)
Аппрув) |
No description provided.