Use url-join to resolve URLs

[JoelMarcey]

Is this a bug report?

No. Enhancement. But could fix unknown or future bugs.

See #745 for context.

https://github.com/facebook/Docusaurus/blob/master/lib/core/Head.js would be a good place to start with for a PR. instead of concatentating URL fragments with +, use the url-join npm package instead.

Have you read the Contributing Guidelines on issues?

Yes

Environment

N/A

Steps to Reproduce

N/A

Expected Behavior

N/A

Actual Behavior

N/A

Reproducible Demo

N/A

[vaibhavsingh97]

@JoelMarcey Can I take this issue? I would love to solve this 😄

[yangshun]

It's yours!

I've added an "issue-claimed" label so that others will know not to start work on the issue. If you change your mind about the issue, no worries! Just let me know so that I can remove the label and free it up for someone else to claim.

I'll check in with you periodically so that we can keep the task updated with the progress.

[vaibhavsingh97]

@JoelMarcey @yangshun Sent PR 😄 #771

[endiliey]

I think we're doing quite good for now and let's not add extra dependency. We are moving to v2 anyway 😄