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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

馃摎 Documentation: Getting Started Links in Readme not valid #8040

Closed
2 tasks done
BtheGit opened this issue Apr 29, 2024 · 3 comments 路 Fixed by #8049
Closed
2 tasks done

馃摎 Documentation: Getting Started Links in Readme not valid #8040

BtheGit opened this issue Apr 29, 2024 · 3 comments 路 Fixed by #8049
Assignees
@BtheGit
Labels
community PR or issues handled by the community members who may need guidance from core good first issue Good for newcomers product / docs Fixes and upgrades for the Appwrite Docs.

Comments

@BtheGit
Copy link
Contributor

BtheGit commented Apr 29, 2024

馃挱 Description

Looks like the getting started links in the readme are linking to non-existent folders in the main repo (eg, https://github.com/appwrite/appwrite/blob/main/docs/quick-starts/web) because they are using relative URLs instead of linking to the hosted docs pages. None of the links work anymore.

Change was introduced in this commit ba7fefc .

馃憖 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

馃彚 Have you read the Code of Conduct?
@stnguyen90 stnguyen90 added good first issue Good for newcomers product / docs Fixes and upgrades for the Appwrite Docs. labels Apr 29, 2024
@stnguyen90
Copy link
Contributor

@BtheGit thanks for creating this issue! 馃檹馃徏 Ya, these should definitely be absolute URLs pointing to our website like https://appwrite.io/docs/quick-starts/web.

@BtheGit
Copy link
Contributor Author

BtheGit commented Apr 29, 2024

Well, I've been hoping to start participating so, a good first issue is great news to me! :)

I'll submit a PR ASAP if nobody else gets there first.

@stnguyen90
Copy link
Contributor

@BtheGit, assigned! Thanks for your interest! 馃檹馃徏

@stnguyen90 stnguyen90 added the community PR or issues handled by the community members who may need guidance from core label Apr 29, 2024
@BtheGit BtheGit mentioned this issue Apr 30, 2024
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BtheGit BtheGit

Labels
community PR or issues handled by the community members who may need guidance from core good first issue Good for newcomers product / docs Fixes and upgrades for the Appwrite Docs.
Projects
1.5 release
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace missing domain paths in README.md BtheGit/appwrite
2 participants
@stnguyen90 @BtheGit