-
Notifications
You must be signed in to change notification settings - Fork 93
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
New contributor table #380
Conversation
@fey Сделал табличку с краткой статистикой по issue#325 . |
@ikhanter давайте как сделаем - если пользователь залогинен, то отображаем кнопку "Compare" (сравнить) |
У вас есть доступ к Хекслетовским курсам? можно глянуть по визуализациям, как лучше для чего подходят https://ru.hexlet.io/courses/analytics-in-business |
т.е. у нас используется другой даже дизайн, там не подробная таблица, а основная инфа в красивом виде |
@fey Сделал, но с рендера авторизоваться не получается, так как authтокены, видимо, не проставил. И согласно DEPLOYMENT.md их для самого веб-сервиса не нужно проставлять, только для заполнения БД в локальном окружении. Прилагаю скринами. На странице, если пользователь авторизован: Фильтр по периоду времени работает. |
Или имелось ввиду сделать в виде графиков? Увы, я пока не силен в matplotlib, а за референс брал уведомление о завершении какого-то конкретного курса, как оформлены баллы и прочее. Могу попробовать в виде столбчатой диаграммы оформить, но она уже как будто реализована и так, в том числе и в виде сравнения, просто на js |
Не, графики можно не делать. |
@fey Сделал. Скрин: |
агга, давайте первую колонку уберем. Мы можем сами значения подкрашивать. Например если левое значение меньше, значит оно красное, а правое - зеленое. Просто колонка-дифф может быть контринтуитивной, ведь плюс или минус зависит от того что из чего вычитаем. |
@fey Изменил: |
Отлично, теперь давайте добавим немного красоты. Сейчас у нас есть селект с периодом. Но он не очень симпотный. Во первых там должны быть какие-то значения. Думаю если мы выбираем период, то должен быть период, например месяц, а не просто прочерк. Если за все время, то так и написано. то есть через симпотичные вкладки. неделя/месяц/за все время Вот тут можно табсы посмотреть По сути у нас каждая вкладка будет своей страницей. Будет немного дублирования, но это нормально. Пока что нужно точно сделать - в демке настроить логин через гитхаб, чтобы я мог войти и потыкать кнопки. |
@fey Поправил, в том числе с замечаниями выше, все работает: |
@sgmdlt глянь, пжлста |
визуально ок. Поправьте онфликты в файлике и скидывайте на ревью |
@fey Сделал |
Так, а давайте еще тест добавим на новую страничку. |
@fey Добавил тест, он проходится, но параллельно падает ошибка, не влияющая на прохождение теста (как и с типами данных даты в тестах, написанных до меня): requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.github.com/users/mintough57 . Я по поводу нее уже писал в других ветках и телеграме, так как не знаю, как от нее избавиться, но она не влияет на прохождение теста |
@sgmdlt посмотри, пожалуйста код, если ок мержи. |
@ikhanter Иван пока в отпуске, можно другие задачки поделать |
Возможно нужно еще добавить перевод страницы |
Add Summary table for check and compare statistics