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

query counter middleware to count number of database queries on each call for better debugging on APIs #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Execut3
Copy link

@Execut3 Execut3 commented Oct 8, 2020

two features added:

1- Query Counter Middleware to count queries on each call of django for better debuggings. it will show a table of read, write and .. query results in console so you can have a fast understanding on how many call is made on each django request call
2- price_beautifier method

@coveralls
Copy link

Coverage Status

Coverage decreased (-13.8%) to 62.015% when pulling e47879b on Execut3:master into 35a42fe on timsavage:master.

@timsavage
Copy link
Owner

timsavage commented Oct 8, 2020

Thank you for the PR request, can you please ensure that all comments follow the same style as the rest of the package.

Please also add test cases and update documentation.

Thanks

@timsavage timsavage added tests-required This item requires test cases enhancement labels Oct 8, 2020
@Execut3
Copy link
Author

Execut3 commented Oct 9, 2020

appreciate your comments. Yes of course will add the parts you mentioned and will send request again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement tests-required This item requires test cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants