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

fix: update logs in deployment app #475

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

7saikat7
Copy link

@7saikat7 7saikat7 commented Jan 12, 2022

Pull Request

What does this PR do?

Fixes #469
Added logging in the deployment app following the best practice

What part does this affect?

  • FrontEnd.
  • BackEnd.
  • Documentation.
  • Other. (Please specify below)

Before submitting

  • Was this discussed/approved via a GitHub issue or slack?
  • Did you read the contributor guideline?
  • Did you ensure that there aren't any other open Pull Requests for the same update/change?
  • Did you make sure the title is self-explanatory and the description concisely explains the PR?
  • Did you make sure your PR does only one thing, instead of bundling different changes together?
  • Did you make sure the code is clean and docstrings have been added or updated as required?
  • Did you make sure the code is linted/formatted locally prior to submission? (using black and/or prettier)
  • Did you make sure to update the documentation with your changes? (if necessary)

PR review

Anyone in the community is free to review the PR once the tests have passed.

Thank you for contributing to AutoDL. We look forward to your continued support.

BackEndApp/deployments/views.py Outdated Show resolved Hide resolved
Co-authored-by: Mcvean Soans <69250084+McTechie@users.noreply.github.com>
@ADI10HERO ADI10HERO changed the title fix: Added and modified logs in deployment app fix: update logs in deployment app Mar 1, 2022
@7saikat7
Copy link
Author

7saikat7 commented Apr 3, 2022

any updates on this sir ?

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.

Add logs in Backend - deployments
2 participants