Skip to content

feat(friends): display common friends on users' pages - #1660

Open
mrcatman wants to merge 5 commits into
OpenVK:masterfrom
mrcatman:common-friends
Open

feat(friends): display common friends on users' pages#1660
mrcatman wants to merge 5 commits into
OpenVK:masterfrom
mrcatman:common-friends

Conversation

@mrcatman

@mrcatman mrcatman commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

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

@veselcraft veselcraft left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

для первого пулл реквеста очень неплохо)

</div>
</div>
</div>
<br/>

@myslivets myslivets Jul 4, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Коммит супер, но "br" здесь тупо лишний)
В остальном всё круто

image

@veselcraft

veselcraft commented Jul 4, 2026

Copy link
Copy Markdown
Member

да, lint бы пофиксить, и можно пушить

@veselcraft

Copy link
Copy Markdown
Member

киря, пофикси линт. иначе мы принять не сможем)))

@crythoughts crythoughts left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

  • нужен метод friends.getMutual
  • предлагаю на странице друзей добавить ссылку типа друзья друзей, но только тех которых у текущего юзера в друзьях нет

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