Skip to content

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.

3 participants