WICG / webmonetization

Proposed Web Monetization standard
https://webmonetization.org
Other
466 stars 152 forks source link

Update links in Resources section #335

Closed huijing closed 1 year ago

huijing commented 1 year ago

Closes #334

This PR fixes the "Page not found" issue with the 2 links in the Resources section. Upon further inspection, it seems that the first 3 links are incorrect and this has been rectified as well.

FYI, the specification, although accessible at https://webmonetization.org/specification.html, is not actually part of the SPA pages generated by Docusaurus, therefore we need to utilise the escape hatch of pathname:// to resolve this (see https://docusaurus.io/docs/advanced/routing#escaping-from-spa-redirects)

🎩 Top-hatting instructions (i.e. human testing that things are not broken)

  1. Pull this branch
  2. Go to http://localhost:3000/ and scroll to the resources section. Verify that all links are working correctly.
netlify[bot] commented 1 year ago

Deploy Preview for webmonetization-preview ready!

Name Link
Latest commit ae66bc5e4554037985499b9e6218ae66ba1bc038
Latest deploy log https://app.netlify.com/sites/webmonetization-preview/deploys/64951d49b9c0260008c8b23b
Deploy Preview https://deploy-preview-335--webmonetization-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

sublimator commented 1 year ago
image

checked as well, LGTM