Skip to content

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