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.

Sign up for GitHub

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

Jump to bottom

feat: All good Docs things under the sun (favicon, oauth2, docs meta) #2855

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: All good Docs things under the sun (favicon, oauth2, docs meta) #2855

wants to merge 1 commit into from
+361 −275

Conversation

worldwidepixel
Copy link

Multiple beneficial additions to Docs site

  • Fixes favicon (expected SVG, got ICO)
  • Adds properly detailed instructions for prospective Docs contributors
  • Improves documentation of Modrinth's Oauth2 capabilities

I'm worried that my code formatter may have accidentally gotten a little overzealous with openapi.yaml, so that may be an issue.

@Erb3
Copy link
Contributor

Erb3 commented Oct 23, 2024

See #2843

@worldwidepixel
Copy link
Author

worldwidepixel commented Oct 23, 2024 •
edited
Loading

I probably should have split my content additions and favicon fixes; This PR is not just to fix the issue #2843 resolves.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@worldwidepixel @Erb3