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

Add log messages for BackEndApp/v1 #337

Open
RusherRG opened this issue Sep 28, 2021 · 7 comments
Open

Add log messages for BackEndApp/v1 #337

RusherRG opened this issue Sep 28, 2021 · 7 comments
Labels
django Backend/Django related work feature New feature or request good first issue Good for newcomers

Comments

@RusherRG
Copy link
Member

Feature Request

Is your feature request related to an existing Issue or a new implementation?

No

Feature Description

Use the configured logger library to add comprehensive and easy to understand log messages in BackendApp/v1/

Possible Solution

import logging
logger = logging.get_logger(__name__)

# add the following wherever necessary
logger.debug("DEBUG MESSAGE")
logger.info("INFO MESSAGE")
logger.warn("WARNING MESSAGE")
logger.error("ERROR MESSAGE")
@RusherRG RusherRG added feature New feature or request good first issue Good for newcomers django Backend/Django related work hacktoberfest labels Sep 28, 2021
@ADI10HERO ADI10HERO added this to To do in Hacktoberfest 2021 Sep 29, 2021
@Aniket-508
Copy link

I want to take up this issue but have some questions!

@Aniket-508
Copy link

Do we have to create a separate logging file in Backend/v1/ which checks for all files simultaneously or implement the predefined library code by importing and checking in individual files?

@RusherRG
Copy link
Member Author

RusherRG commented Oct 1, 2021

So we already have the configured logger here. All you have to do is import the logging library and add info, error, warn, etc.. logs wherever needed in the scripts residing in BackEndApp/v1.

You can use the following as reference

import logging
logger = logging.get_logger(__name__)

# add the following wherever necessary
logger.debug("DEBUG MESSAGE")
logger.info("INFO MESSAGE")
logger.warn("WARNING MESSAGE")
logger.error("ERROR MESSAGE")

@codewith-arijit
Copy link

codewith-arijit commented Oct 1, 2021

I would also like to take up this issue. Can multiple users send their PR? Can I be assigned for it?

@McTechie
Copy link
Contributor

McTechie commented Oct 5, 2021

@Aniket-508 @codewith-arijit any one willing to take up this issue? (You would be assigned to it)

@prajwalnayak7
Copy link

@McTechie I have created the PR. Please review and assign this.

@McTechie
Copy link
Contributor

McTechie commented Oct 5, 2021

It would've been appreciated if you discussed here before creating the PR, but no worries we'll have a look at it.
P.s. For tackling other issues, kindly comment on the Respective Issue before creating a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
django Backend/Django related work feature New feature or request good first issue Good for newcomers
Projects
Development

Successfully merging a pull request may close this issue.

5 participants