Skip to content
This repository was archived by the owner on Apr 19, 2023. It is now read-only.
This repository was archived by the owner on Apr 19, 2023. It is now read-only.

Refactoring for better readability and practice. #6

Closed
@chand1012

Description

@chand1012

Recently a friend of mine helped me out with a code review. Here are the suggestions that need fixed.

image

  • The above issue.
  • The variables client_id and client_secret should not be set to empty strings, should be set to None. It's faster and better practice.
  • The default string "oauth.json" should be moved to a variable at the start of the file (or in __init__.py).

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions