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

Documentation Improvements #6114

Merged
merged 4 commits into from
Sep 7, 2021
Merged

Documentation Improvements #6114

merged 4 commits into from
Sep 7, 2021

Conversation

ApCoder123
Copy link
Member

@ApCoder123 ApCoder123 commented Sep 5, 2021

I've made a couple of modifications to the documentation:

  • Consistency (Phrase "two factor authentication", rather than a mixture of "two factor auth" and "Two Factor Authentication")
  • Remove outdated references (there were a couple of references to the old YAML schema in CONTRIBUTING.md)
  • Move new categories to end of contributing guidelines (the majority of contributors are looking to add sites)
  • A couple of minor language changes & typo fixes
  • Mark v2 API as deprecated
  • Remove Gitter link (it doesn't seem to have been used properly since 2019)

@ApCoder123 ApCoder123 added the enhancement Issue/PR contains enhancements to the overall code of the site. label Sep 5, 2021
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@phallobst phallobst merged commit e9336b6 into 2factorauth:master Sep 7, 2021
@ApCoder123 ApCoder123 deleted the docs branch September 7, 2021 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue/PR contains enhancements to the overall code of the site.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants