Skip to content

Gunicorn nginx branch - #3

Open
virsi wants to merge 4 commits into
mainfrom
gunicorn_nginx_branch
Open

Gunicorn nginx branch#3
virsi wants to merge 4 commits into
mainfrom
gunicorn_nginx_branch

Conversation

@virsi

@virsi virsi commented Dec 14, 2025

Copy link
Copy Markdown
Owner

Выполнено ДЗ№6 - Добавлен Gunicorn и Nginx. ДЗ№4-5 реализованы в предыдущем ПР (убежал вперед)

Comment thread nginx/cache/9/87/29ce5a380cc94b14e4a271d8d371b879 Outdated
Comment thread nginx/nginx.conf
Comment thread nginx/nginx.conf
@sh4rkizz

Copy link
Copy Markdown
изображение

Comment thread nginx/nginx.conf
@sh4rkizz

sh4rkizz commented Jan 8, 2026

Copy link
Copy Markdown

сейчас vote_question выполняет редирект return redirect('questions:list'), это не ок поведение для AJAX вьюхи [коей она должна являться]

Должен возвращаться json-ответ

@sh4rkizz

sh4rkizz commented Jan 8, 2026

Copy link
Copy Markdown

Для vote answer при нажатии стреляет такая приколюха, тоже нужно чтобы возвращался json, который клиентом как-то парсится

изображение

@sh4rkizz

sh4rkizz commented Jan 8, 2026

Copy link
Copy Markdown

Аналогично, ожидаем что приходит только ajax, нет смысла проверять

изображение

@sh4rkizz

sh4rkizz commented Jan 8, 2026

Copy link
Copy Markdown
def get_avatar(self):
    if self.avatar:
        return self.avatar.url
    return 'static/img/avatar-placeholder.jpg'

вот эта конструкция должна быть другой, реализуется метод модели (например в User), который обрабатывает наличие/отсутствие картинки

изображение

@sh4rkizz

sh4rkizz commented Jan 8, 2026

Copy link
Copy Markdown

если я под незарегом не могу оставлять лайки, то лучше скрывать кнопки

изображение

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