Skip to content

Первая итерация #2

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Первая итерация #2

wants to merge 6 commits into from

Conversation

FArthur-cmd
Copy link
Owner

Добавлено считывание валют с сайта центробанка и их сохранение в базу данных(чтобы избежать повторных скачиваний), Добавлены тесты для корректности загрузки и работы моделей. Критерии запуска тестов и самого сайта указаны в readme.

@Mirinda2516
Copy link
Collaborator

при нажатии на Турецкая лира(TRY) 2001-04-03 падает с ошибкой MultiValueDictKeyError at /Турецкая лира/2001-04-03/show_wallet/
Проверь, работает ли у тебя, возможно, я не доустанавливала какие-то пакеты (хотя, кажется, ошибка не об этом), тк на винде их установка падает на каждом 2-ом
по коду всё хорошо, единственное, не всегда понимаю логику твоих переносов на новую строку (переносить после знака "=" - не лучшая идея, как и дважды переносить маленький хвостик типа такого
wallet = Wallet_indentificator.objects.get(wallet_name=
valute.find(
'name').text)

@FArthur-cmd
Copy link
Owner Author

Добавил фикс для данной ошибки. Причина: внезапно возникшая повторная дата. Решение: добавил функцию, которая чистит повторную дату, если вдруг ее находит. Добавил тест для этой функции, что она корректно выполняет свои действия

@FArthur-cmd
Copy link
Owner Author

По поводу переносов - я доверился среде разработке и встроенному в нее пеп8. Увы она часто творит такие вещи и в процессе написания кода я уже не замечаю этого. Поправил в последнем коммите

@Mirinda2516
Copy link
Collaborator

ок, хорошо)
Проект принят.

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