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: relative links in website to absolute links #923

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

felixmosh
Copy link
Contributor

@felixmosh felixmosh commented Nov 4, 2020

Maintainer edit:

Closes #844

@vercel
Copy link

vercel bot commented Nov 4, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/formium/tsdx/qfmj3i24j
✅ Preview: https://tsdx-git-patch-1.formium.vercel.app

@agilgur5 agilgur5 changed the title Fix website links fix: relative links in website to absolute links Nov 4, 2020
@agilgur5
Copy link
Collaborator

agilgur5 commented Nov 4, 2020

Supersedes #844 with a valid solution to another issue with #765 , though my remarks in #844 (comment) still stand, especially as the website has no versioning. The website is also a bit outdated now because it was never synced with the README (c.f. #844 (comment), #848).

This is still a good change though, so will accept this as incremental improvement even though it does not solve all the problems.

Copy link
Collaborator

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

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

URLs are correct and I've tested that they work in the preview site 👍

Tests seem to be timing out again... will have to re-run those

@agilgur5 agilgur5 added the scope: docs Documentation could be improved. Or changes that only affect docs label Nov 4, 2020
@agilgur5
Copy link
Collaborator

agilgur5 commented Nov 4, 2020

Another test timeout but only 1 this time. I'll merge since this is quite safe to change, doesn't affect any actual code, and was manually tested.

@agilgur5 agilgur5 merged commit 6078acb into jaredpalmer:master Nov 4, 2020
@agilgur5
Copy link
Collaborator

agilgur5 commented Nov 4, 2020

@allcontributors please add @felixmosh for docs. Thanks for the contribution

@allcontributors
Copy link
Contributor

@agilgur5

I've put up a pull request to add @felixmosh! 🎉

@felixmosh felixmosh deleted the patch-1 branch November 5, 2020 07:35
paul-vd pushed a commit to EezyQuote/tsdx that referenced this pull request Dec 1, 2020
- relative links in website were not working as the files can't be viewed
  on the site
  - these were erroneously copy+pasted from the README, where the relative
    links work by pointing to files in the repo
    - those relative links don't work on the website

- use absolute links instead to the main fork of the repo's `master` branch
  - this has a few caveats now, in that the README has a bit more desync from
    the website now and that, unlike the README, these point to a specific
    fork and branch
    - the README points to the same version; regardless if forked or on a
      different branch, it'll point to that branch's code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: docs Documentation could be improved. Or changes that only affect docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants