fixes issue #1169

[gaurav879]

🚀 🚀 Pull Request

Checklist:

• My code follows the style guidelines of this project and the Contributing document
• I have commented my code, particularly in hard-to-understand areas
• I have kept the coverage-rate up
• I have performed a self-review of my own code and resolved any problems
• I have checked to ensure there aren't any other open Pull Requests for the same change
• I have described and made corresponding changes to the relevant documentation
• New and existing unit tests pass locally with my changes

Changes

Added 3 chances for the user to login using CLI

[CLAassistant]

All committers have signed the CLA.

[davidbuniat]

Thanks @gaurav879 for the contribution! and I agree that 3x authentication would be a better UX. Just curious what motivated you the problem?

[gaurav879]

@davidbuniat, you're welcome. I'm new to open source, and I'm working on some good first issues and familiarising myself with the codebase. The above problem was simpler, and I felt confident in my ability to address it. I intend to work on and contribute to more issues in the future.

[AbhinavTuli]

Hey @gaurav879! Everything looks good, I've left a couple of small comments and I can approve once those are addressed. Looking forward to seeing more contributions from you 🚀

[gaurav879]

Hey, @AbhinavTuli I have made the requested changes. Sorry for the delay from my side.

[AbhinavTuli]

Looks good. One minor change would be to switch from print to click.echo for consistency.

[mikayelh]

Yay, @gaurav879! Thanks for contributing. Hit us up in community slack for some swag. :)