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

#60 - Change button to link. #62

Merged
merged 1 commit into from
Oct 12, 2021

Conversation

exegeteio
Copy link
Contributor

Changing the Sign In button to a link so that a GET request is created as opposed to a POST. This prevents an error on loading the sign in page for the first time.

Can this be marked as "too simple for Hacktoberfest" ? I don't think this should count.

Changing the Sign In button to a link so that a GET request is created as opposed to a POST.  This prevents an error on loading the sign in page for the first time.
@ChaelCodes
Copy link
Owner

Can this be marked as "too simple for Hacktoberfest" ? I don't think this should count.

LOL! I'll merge it after you've made your four, but I don't think there's a way to exclude it.

@ChaelCodes
Copy link
Owner

Please include the issue it closes with closes # Thanks! Also, I appreciate how clean and well-written that issue was.

@ChaelCodes
Copy link
Owner

Also, it clearly passes Hacktoberfest quality standards.

@ChaelCodes
Copy link
Owner

@exegeteio You ready for me to merge this?

@ChaelCodes ChaelCodes merged commit 878bb87 into ChaelCodes:main Oct 12, 2021
@ChaelCodes ChaelCodes added this to the Hacktoberfest 2021 milestone Nov 1, 2021
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.

2 participants