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

Обновил логику добавления в контекст последних 10 пулл реквестов и проблем на главной странице #422

Merged
merged 2 commits into from
Jun 7, 2024

Conversation

thiSSSnake
Copy link

Обновил логику добавления в контекст последних 10 пулл реквестов и проблем на главной странице в home.py, добавил новый шаблон в compontents/latest_issues_and_pr.html, изменил код в JS скрипте.

…he context on the main page home.py , added a new template to compontents/latest_issues_and_pr.html , changed the code in the JS script.
@fey
Copy link
Collaborator

fey commented Jun 5, 2024

@thiSSSnake задеплойте, пожалуйста, демку.

@@ -35,7 +35,7 @@ function time_note(data_type) {


document.addEventListener('DOMContentLoaded', start('top-week'));
document.addEventListener('DOMContentLoaded', start('latest-for-week'));
document.addEventListener('DOMContentLoaded', start('latest-in-recent-times'));
Copy link
Collaborator

Choose a reason for hiding this comment

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

а нам это действительно нужно? По идее в бутстрапе табы работают из корробки. И по идее должен по дефолту просто открываться один там.
Жду демку, посмотрю, что внутри.

Copy link
Author

@thiSSSnake thiSSSnake Jun 5, 2024

Choose a reason for hiding this comment

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

@fey При деплое столкнулся с проблемой, сервер отдает 500 ошибку, хотя деплой успешно прошел да и локально сервер работает, allowed_hosts в конфиге не трогал, деплоил по инстуркции в репозитории...буду смотреть сегодня в чем дело :/
image

Copy link
Author

@thiSSSnake thiSSSnake Jun 6, 2024

Choose a reason for hiding this comment

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

@fey проблема была в том что я новый шаблон в коммит не добавил -__-. Ссылка на демо -- https://hexlet-friends-w4ta.onrender.com

@fey
Copy link
Collaborator

fey commented Jun 7, 2024

посмотрел. ну навертели тут всякого конечно когда-то))

@fey fey merged commit 4ddf012 into Hexlet:main Jun 7, 2024
1 check passed
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.

2 participants