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

#543 Add log to ya aviso view. #544

Merged
merged 1 commit into from
Aug 25, 2018
Merged

#543 Add log to ya aviso view. #544

merged 1 commit into from
Aug 25, 2018

Conversation

duker33
Copy link
Contributor

@duker33 duker33 commented Aug 24, 2018

And change order list fields set at admin panel

Closes #543

@duker33 duker33 self-assigned this Aug 24, 2018
search_fields = ['name', 'email', 'phone']
list_display_links = ['name']


Copy link
Contributor

Choose a reason for hiding this comment

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

this blank line is not required

Copy link
Contributor Author

@duker33 duker33 Aug 25, 2018

Choose a reason for hiding this comment

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

@artemiy312 , that's what pycharm told me about it:
image

It's strange, coala should check it. This issue was fixed at version 2.2.0 of pycodestyle. But our coala uses version pycodestyle==2.3.1
PyCQA/pycodestyle#400

UPD: it's because pycodestyle_ignore = E305,W503 string. Created issue #548.

UPD2: This string is about pep8, so i leave it here

@duker33 duker33 merged commit 8f7127f into master Aug 25, 2018
@duker33 duker33 deleted the 543_ya_kassa_bug branch August 25, 2018 08:51
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