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

Fix of Broken "Installation" Link in README.md #6432

Closed
wants to merge 1 commit into from
Closed

Fix of Broken "Installation" Link in README.md #6432

wants to merge 1 commit into from

Conversation

imperialc0der
Copy link

@imperialc0der imperialc0der commented Oct 2, 2023

What does this PR do?

This PR will fix the Broken "Installation" link at "[Upgrade from an Older Version]"

###Description

In the README.md file, under the Installation Section there is a link (please see the screenshot below)
image

When you click on the Installation Docs, It redirects you to an url "https://appwrite.io/docs/installation" . But this URL shows 404 error. The valid URL is "https://appwrite.io/docs/advanced/self-hosting" . Which i have attached in my Commit.

BEFORE THE FIX:
image

AFTER THE FIX:
image

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@Susmita-Dey
Copy link

Susmita-Dey commented Oct 3, 2023

Hey @imperialc0der Please write the issue heading and description properly as it's not releaving properly what you have changed inside that file.

You may use this guide as a reference and update the description as per the given template.

@imperialc0der
Copy link
Author

Hey @Susmita-Dey i have updated my main comment. Please check now.

@Susmita-Dey
Copy link

Hey @imperialc0der Update the issue title properly aligning with your PR

@imperialc0der imperialc0der changed the title Update README.md Fix of Broken "Installation" Link in README.md Oct 3, 2023
@imperialc0der
Copy link
Author

@Susmita-Dey Done.

Copy link

@Susmita-Dey Susmita-Dey left a comment

Choose a reason for hiding this comment

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

LGTM

@stnguyen90
Copy link
Contributor

We are taking part of this as #6437. If you wish to contribute as part of Hacktoberfest, please look for issues labeled "hacktoberfest" and do not raise PRs for things you are not assigned.

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.

3 participants