-
Notifications
You must be signed in to change notification settings - Fork 29
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
turfjs.org/docs/ is broken #124
Comments
Correct, I think a redirect would be ideal. http://turfjs.org/docs/ |
@rowanwins Any thoughts on how we can handle that with VueJS?.. 🤔 |
Think I should be able to manage it |
If that redirect could retain the |
Roger that, will see how I go, thanks @andrewharvey . I did manage to kind of get things tidied last night but then got distracted trying to fix the bundle size :) |
@andrewharvey 👍 For sure, we should preserve the previous URL references of TurfJS that way there's less broken links in the wild. |
http://turfjs.org/docs/ currently returns a 404. This is breaking a lot of links in the wild. |
Yeah sorry @andrewharvey - still trying to find time to get on to this, the joy of trying to prioritise open-source work at 10pm :) |
@rowanwins I understand and realise we're all volunteers here. Could we re-instate the old docs until this is resolved, that takes the time pressure off this and means links aren't broken? Then once this is fixed you can roll out the new docs once again. |
Let me take a look tonight @andrewharvey and I'll see if I can get it sorted. Unfortunantly even reverting is challenging because the old docs were for pre-v5 which means they didn't say anything about the options object. |
@rowanwins I did some changes to fix this... hopefully it didn't break anything.. 🤞 (unfortunately I'm going to bed and I won't be able to test this until tomorrow) |
The website should be working as intended again: The following links work: |
Thanks for fixing this! One minor point, the docs used to accept lower case but now they don't so links in the wild like |
The docs for turf used to be at http://turfjs.org/docs/ right? Now that's broken, http://turfjs.org/docs works, so perhaps this needs a redirect?
The text was updated successfully, but these errors were encountered: