-
Notifications
You must be signed in to change notification settings - Fork 7
516 log system #525
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
base: develop
Are you sure you want to change the base?
516 log system #525
Conversation
erdivartanovich
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add more proper information here, maybe add referred by ?
f144430#diff-c8d46fac01f8d0ee23cecf0ba91adfceR91
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is the purpose of os and sqlalchemy desc import, i did not see you used it anywhere in your code ?
285f8ea
erdivartanovich
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change the position of logging after the ticket is crated and should add the ticket information in the log message
7f19d18
erdivartanovich
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The log should display payment status information if it failed or success
60e4287
erdivartanovich
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The log should add information off order id that affected by each tansaction
d36fe3e
erdivartanovich
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also add information about the ticket that affected by this transaction
94eba49
|
fixed mr. @erdivartanovich |
#516
Breakdown: