Skip to content

Sprint 1: Реализована базовая версия GilgaChat — лейауты, формы, страницы ошибок и деплой#1

Merged
maximste merged 101 commits into
mainfrom
sprint_1
Mar 9, 2026
Merged

Sprint 1: Реализована базовая версия GilgaChat — лейауты, формы, страницы ошибок и деплой#1
maximste merged 101 commits into
mainfrom
sprint_1

Conversation

@maximste
Copy link
Copy Markdown
Owner

@maximste maximste commented Mar 8, 2026

Sprint 1: базовая версия GilgaChat

Что сделано

Роутинг и точка входа

  • Роутинг по hash (#auth, #register, #profile, #messenger, #404, #500)
  • Точка входа в src/main.ts: подписка на DOMContentLoaded и hashchange, выбор layout и экрана по window.location.hash

Лейауты

  • MainLayout — шапка (GilgaChat, ссылка «Назад»), область контента для авторизации, профиля и страниц ошибок
  • MessengerLayout — сайдбар (поиск, личные чаты, группы, текущий пользователь - в виде заглушек) и основная область с заглушкой «Чат не выбран» (NoChatStub)

Экраны и компоненты

  • Формы входа и регистрации с логотипом
  • Страница профиля (карточка и демо-данные)
  • Страницы ошибок 404 и 500
  • Базовые UI-компоненты: Button, Input, Label, Link, FormField

Стили и ресурсы

  • Общие стили в src/styles/ (цвета, типографика, отступы, переменные)
  • Логотип в public/images/logo.png

Деплой и документация

  • Деплой на GitHub Pages из ветки deploy (workflow в .github/workflows/)
  • Обновлён README: демо, стек, структура проекта, инструкции по установке и деплою, примечание про Netlify

maximste added 19 commits March 8, 2026 11:24
Copy link
Copy Markdown

@mikebars1995 mikebars1995 left a comment

Choose a reason for hiding this comment

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

Здравствуйте. (Нужно развернуть общий комментарий ↓)

Работа проделана огромная

Что сделано отлично:

  • Readme хорошо оформлен
  • Проект задеплоен и отлично работает
  • Отлично, что не забыли про .gitignore
  • Отлично, что ограничиваете версию node
  • Отлично, что нет EOF ошибок в гите
  • Хорошая структура папок и файлов
  • Отлично, что цвета вынесены в css-константы
  • Отлично, что не забываете про alt в тегах img
  • Отлично, что не забываете про тег main для главной части страницы

Что нужно исправить:

  • По заданию 1го спринта нужно давать инпутам конкретные атрибуты name, чтобы в 3м спринте не менять ничего при подключении к Api. Скрин из задания https://disk.yandex.ru/i/lSPqeGrAa20ayQ
  • Нужно сверстать еще страницу редактирования данных профиля, она должна быть с формой и инпутами, у инпутов должны быть атрибуты name по заданию

Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.

Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.

Комментарии Можно лучше не обязательны к исправлению прямо сейчас, это рекомендации

Удачного рефакторинга кода.

const nameInput = new Input({
id: 'userName',
type: 'text',
name: 'name',
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Нужно исправить

По заданию 1го спринта нужно давать инпутам конкретные атрибуты name, чтобы в 3м спринте не менять ничего при подключении к Api. Скрин из задания https://disk.yandex.ru/i/lSPqeGrAa20ayQ

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Забыл, что это требование в тренажере описано, да и в api можно было посмотреть, да. Исправил.

@maximste maximste requested a review from mikebars1995 March 8, 2026 14:10
Copy link
Copy Markdown

@mikebars1995 mikebars1995 left a comment

Choose a reason for hiding this comment

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

Поздравляю! Ваша работа принята.

Вы отлично потрудились.

Удачного дальнейшего обучения.

@maximste maximste merged commit 77b83e9 into main Mar 9, 2026
1 check passed
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