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

Update Docs #67

Merged
merged 2 commits into from
Apr 23, 2024
Merged

Update Docs #67

merged 2 commits into from
Apr 23, 2024

Conversation

Hamster45105
Copy link
Contributor

Just some small changes to the docs, including:

  • Fix page edit button not working
  • Rename NodeJS to Node.js
  • Add pip and npm links to install section
  • Other small changes

Also, as I mentioned in my previous PR:

I would recommend updating the homepage link on the GitHub repo from the current one:
screenshot

to the docs homepage (https://jeanextreme002.github.io/FlightRadarAPI/) to make it easier for people to get to the docs. There are links to both PyPI and NPM on the docs homepage.

It would also be worth deleting the wiki as it is not really needed any more.

@Hamster45105 Hamster45105 force-pushed the main branch 2 times, most recently from 053b422 to 82f4dbb Compare April 20, 2024 07:43
Copy link
Owner

@JeanExtreme002 JeanExtreme002 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As for the homepage link on GitHub, I'll leave it as the PyPi link because I want to highlight it as the primary source for download.

README.md Show resolved Hide resolved
@JeanExtreme002 JeanExtreme002 merged commit ac6479d into JeanExtreme002:main Apr 23, 2024
9 checks passed
@JeanExtreme002
Copy link
Owner

Merged! Thanks again for contributing to the project!! ☺️

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