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

issue #214: Compare your statistics with other users #224

Merged
merged 6 commits into from Feb 3, 2023

Conversation

yuriy-kormin
Copy link
Contributor

@yuriy-kormin yuriy-kormin commented Dec 28, 2022

Screenshot 2022-12-28 at 17 27 01

Screenshot 2022-12-28 at 17 27 10

Итоговые данные

Добавлена строка ИТОГО в конце таблицы

Сравнение

На странице создается input type="hidden" в котором в формате json хранятся данные всех пользователей. После с помощью javascript обновляются данные для сравнения.

Поля, которые у своего пользователя отличаются подсвечиваются с помощью bootstrap style

У меня почти нет опыта в javascript, посему если скрипт (находится в /static/js/contrib_compare) не очень оптимальный - можно попросить кого нить переписать.

@yuriy-kormin yuriy-kormin marked this pull request as ready for review December 28, 2022 13:49
@emp7yhead
Copy link
Contributor

@yuriy-kormin посмотри ошибки линтера, пожалуйста

@acidmaksim
Copy link
Contributor

@yuriy-kormin тыц

@emp7yhead
Copy link
Contributor

@dzencot попрошу тебя посмотреть js?

@emp7yhead
Copy link
Contributor

@yuriy-kormin а ты где-нибудь разворачивал проект, чтобы можно было бы посмотреть?

@yuriy-kormin
Copy link
Contributor Author

только локально. ну можно попробовать docker образ собрать и залить на railway.
так а что там смотреть - в PR поправлен один файл, добавлен скрипт , остальное шаблоны

static/js/contrib_compare.js Outdated Show resolved Hide resolved
static/js/contrib_compare.js Outdated Show resolved Hide resolved
static/js/contrib_compare.js Outdated Show resolved Hide resolved
@emp7yhead
Copy link
Contributor

только локально. ну можно попробовать docker образ собрать и залить на railway.

Давай на railway развернем, чтобы посмотреть как работают изменения

@yuriy-kormin
Copy link
Contributor Author

никак не получается собрать пакет на railway. нужна помощь больно мудреная установка

@yuriy-kormin yuriy-kormin requested review from dzencot and removed request for emp7yhead February 3, 2023 09:08
@emp7yhead emp7yhead merged commit 064b750 into Hexlet:main Feb 3, 2023
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

4 participants