-
Notifications
You must be signed in to change notification settings - Fork 16
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(ui): added broken documentation link in login page #51
fix(ui): added broken documentation link in login page #51
Conversation
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 use a tabs (4spaces) indentation. Your PR is changing the indentation for the entire file
Right!, I realized about that, but I was not sure what is the style guide in the project, others files like |
Good point. For this file, let's keep the same indentation that's already there. In a way that your PR would show only your actual changes. As a separate issue, would you be interested in working on it and consolidate? The repo already uses Eslint. So it should be a matter of adding prettier and configure it properly. Would you be interested in it? |
Sounds good! |
@dev-bre updated!! take a look now! |
@acollazog11 looks good good idea on replacing the button with a link! |
No description provided.