-
Notifications
You must be signed in to change notification settings - Fork 1
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
Features/description of sections #23
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Backend/src/backend/apps/general/models.py
Lines 17 to 22 in 7bca28d
title = models.TextField( | |
verbose_name="Заголовок", max_length=55, null=False | |
) | |
description = models.TextField( | |
verbose_name="Текст", max_length=160, null=False | |
) |
значения max_length лучше вынести в constants.py как константы, чтобы потом легче было вносить изменения, при необходимости или одну и туже константу использовать несколько раз.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не плохо было бы доработать. На странице могут выводиться как одна секция, так и много. При данной реализации фронты будут вынуждены выполнять столько запросов к бэку, а бэк к БД, сколько секций на странице. Желательно реализовать так, чтобы была возможность получить список секций для каждой конкретной страницы одним запросом.
И свяжи ПР с issue через "Development" в столбце справа. |
…w), убрала max_length в константы
No description provided.