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

added UserInfo component #5

Merged
merged 5 commits into from Aug 10, 2022
Merged

added UserInfo component #5

merged 5 commits into from Aug 10, 2022

Conversation

KirilllAll
Copy link
Contributor

  1. Добавил компонент для отображения информации о пользователе
  2. Добавил компонент для отображения сообщения в случае, если информация не доступна
  3. Добавил страницу в папку ssr/users для отображения информации о пользователе.

Copy link
Contributor

@Hubble999 Hubble999 left a comment

Choose a reason for hiding this comment

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

Папка .Idea кажется случайно попала в коммит, наверное в gitignore надо добавить ее

Hubble999
Hubble999 previously approved these changes Jul 26, 2022
Copy link
Contributor

@Hydrock Hydrock left a comment

Choose a reason for hiding this comment

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

Папку .idea нужно удалить
Это довольно серьезная оплошность - нужно проверять что попадает в index гита (зеленое) и что потом закоммитится

@AksenovKirill @Hubble999 обратите внимание

@Hydrock
Copy link
Contributor

Hydrock commented Jul 26, 2022

@Hubble999 ты заапрувил, но в PR есть проблемы - обращай на это внимание

@KirilllAll
Copy link
Contributor Author

@Hydrock @Hubble999 Добавил в gitignore папку и ее из проекта, сделал новый коммит и сделал push. Надеюсь это поможет. Спасибо!

@Hydrock
Copy link
Contributor

Hydrock commented Jul 26, 2022

Нужно зарезолвить конфликты

@KirilllAll KirilllAll requested a review from Hydrock July 27, 2022 18:51
Hydrock
Hydrock previously approved these changes Jul 28, 2022
Copy link
Contributor

@Hydrock Hydrock left a comment

Choose a reason for hiding this comment

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

Есть небольшой совет

yarn.lock Outdated
version "7.18.9"
resolved "https://registry.yarnpkg.com/@babel/runtime-corejs3/-/runtime-corejs3-7.18.9.tgz#7bacecd1cb2dd694eacd32a91fcf7021c20770ae"
integrity sha512-qZEWeccZCrHA2Au4/X05QW5CMdm4VjUDCrGq5gf1ZDcM4hRqreKrtwAn7yci9zfgAS9apvnsFXiGBHBAxZdK9A==
"integrity" "sha512-qZEWeccZCrHA2Au4/X05QW5CMdm4VjUDCrGq5gf1ZDcM4hRqreKrtwAn7yci9zfgAS9apvnsFXiGBHBAxZdK9A=="
Copy link
Contributor

Choose a reason for hiding this comment

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

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

эта проблема дейстивтельно странная - обсуждается тут yarnpkg/yarn#4379
но ее нужно решить иначе мы так и будем перегенеривовать этот файл

если нужно давайте созвонимся

@AksenovKirill @Hubble999 Обратите внимание

Copy link
Contributor

Choose a reason for hiding this comment

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

Буду иметь ввиду, спасибо

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Переустановил yarn

yarn.lock Outdated
@@ -1794,4 +1794,4 @@ wrappy@1:
yallist@^4.0.0:
version "4.0.0"
resolved "https://registry.yarnpkg.com/yallist/-/yallist-4.0.0.tgz#9bb92790d9c0effec63be73519e11a35019a3a72"
integrity sha512-3wdGidZyq5PB084XLES5TpOSRA3wjXAlIWMhum2kRcv/41Sn2emQ0dycQW4uZXLejwKvg6EsvbdlVL+FYEct7A==
integrity sha512-3wdGidZyq5PB084XLES5TpOSRA3wjXAlIWMhum2kRcv/41Sn2emQ0dycQW4uZXLejwKvg6EsvbdlVL+FYEct7A==
Copy link
Contributor

Choose a reason for hiding this comment

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

Тебе нужно настроить твой редактор чтобы по сохранению файла - в конец файла добавлялась пустая строка - это хорошая практика в программировании - так два файла не слипнутся при конкатенации

Copy link
Contributor Author

Choose a reason for hiding this comment

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

настроил

Hubble999
Hubble999 previously approved these changes Aug 2, 2022
@KirilllAll KirilllAll dismissed stale reviews from Hubble999 and Hydrock via d4650e3 August 2, 2022 18:41
Hubble999
Hubble999 previously approved these changes Aug 2, 2022
@KirilllAll
Copy link
Contributor Author

Решил конфликты в yarn.lock

@Hubble999 Hubble999 self-requested a review August 2, 2022 18:57
@KirilllAll
Copy link
Contributor Author

  1. Переустановил yarn.
  2. Проверил PR .

@KirilllAll KirilllAll merged commit 7c767f7 into main Aug 10, 2022
@Hydrock Hydrock deleted the WH-69 branch January 14, 2023 17:57
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.

None yet

3 participants