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

feat: добавлено API для leaderbord #60

Merged
merged 3 commits into from
Sep 22, 2022

Conversation

xczdenis
Copy link
Contributor

No description provided.

Copy link
Contributor

@bestia21v bestia21v left a comment

Choose a reason for hiding this comment

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

Денис, крутая работа) Классно, что заложился на будущие доработки) Оставил пару минорных замечаний)

src/pages/Game/Game.tsx Show resolved Hide resolved
src/pages/Leaderboard/Leaderboard.tsx Show resolved Hide resolved
@axperien
Copy link
Contributor

Круто! 💣
Но у меня результат перезаписывается, если даже он ниже лучшего)
И я бы добавил проверку на наличие лидеров, чтобы не выглядела страничка пустой, если вообще никаких результатов нет, либо например только 3 всего

Copy link
Contributor

@AntLer-24rus AntLer-24rus left a comment

Choose a reason for hiding this comment

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

Круто! 🔥 Как же хорошо что ты решил написать свое API это еще ближе двигает нас к собственной авторизации и отвязке от yandex API. Оставил пару комментов! Супер проделанная работа!

src/server/routes/api/index.ts Show resolved Hide resolved
src/services/leaderboardService.ts Show resolved Hide resolved
@bestia21v bestia21v merged commit 6600c98 into DVAA-team:main Sep 22, 2022
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