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

feat: Добавлена информация о стоимости консультации. #79

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

ravimb06
Copy link
Collaborator

@ravimb06 ravimb06 commented Mar 9, 2023

No description provided.

@ravimb06 ravimb06 requested a review from bmu0 March 9, 2023 10:07
@@ -8,7 +8,12 @@


async def meetings_main_menu(update: Update, context: ContextTypes.DEFAULT_TYPE) -> int:
text = "Выберите, что нужно сделать:"
text = (
"Стоимость консультации психолога (1 час):\n"
Copy link
Collaborator

Choose a reason for hiding this comment

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

50 минут)

@@ -150,6 +150,7 @@ async def inner(update: Update, context: ContextTypes.DEFAULT_TYPE) -> str:
text += f"\nФормат записи: {meeting_format}"
text += f"\nПсихолог: {timeslot.profile.first_name} {timeslot.profile.last_name}"
text += f"\nДата: {timeslot.date_start}"
text += f"\nСтоимость консультации: {MEETING_PRICE} р."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Цену нужно писать только для тех кто уже побывал на 5 встречах, надо добавить проверку количества прошедших встреч

text = "Выберите, что нужно сделать:"
text = (
"Стоимость консультации психолога (1 час):\n"
f"Сейчас {MEETING_PRICE} р.\n"
Copy link
Collaborator

Choose a reason for hiding this comment

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

То же самое - надо выводить только для прошедших 5 встреч

@@ -88,6 +88,7 @@ async def inner(update: Update, context: ContextTypes.DEFAULT_TYPE) -> str:
text += f"\nФормат записи: {meeting_format}"
text += f"\nПсихолог: {timeslot.profile.first_name} {timeslot.profile.last_name}"
text += f"\nДата: {timeslot.date_start}"
text += f"\nСтоимость консультации: {MEETING_PRICE} р."
Copy link
Collaborator

Choose a reason for hiding this comment

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

Наверное тут нажно добавить эту проверку, а из первичной записи вообще убрать - первая запись точно не шестая

@ravimb06 ravimb06 requested a review from bmu0 March 9, 2023 20:14

context_manager.set_timeslot(context, timeslot)

text = "Давайте все проверим:\n"
text += f"\nФормат записи: {meeting_format}"
text += f"\nПсихолог: {timeslot.profile.first_name} {timeslot.profile.last_name}"
text += f"\nДата: {timeslot.date_start}"
if number_of_user_meetings >= 5:
Copy link
Member

Choose a reason for hiding this comment

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

Предлагаю, 5 вынести в settings и установить значение по умолчанию. В переменные окружения выносить не нужно.
Почему в settings, а не constants? Данное значение влияет на бизнес логику. Стоимость и продолжительность консультации не влияют на логику работы, и являются просто постоянными значениями, поэтому они уходят в constants.

username = update.message.chat.username
user = user_service_v1.get_user(username=username)
number_of_user_meetings = len(schedule_service_v1.get_meetings_by_user(user=user.id))
text = "Длительность консультации психолога (50 мин.):\n"
Copy link
Member

Choose a reason for hiding this comment

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

Продолжительность и обычную цену лучше вынести в constants.

@ravimb06 ravimb06 requested a review from ADesBiysk March 10, 2023 15:03
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.

3 participants