Delete duplicate links data file

[karnikaavelumani]

Fix #756.

[netlify]

✅ Deploy Preview for acmcsuf ready!

Name	Link
🔨 Latest commit	42023b1
🔍 Latest deploy log	https://app.netlify.com/sites/acmcsuf/deploys/6418df6447ee250008ace519
😎 Deploy Preview	https://deploy-preview-816--acmcsuf.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[jaasonw]

🍛🍣🐖😶🍕♑🈹🐒🐆🦬🍕🍔🍟😮🍟🥨🥡🍤🥩🚞🍕🚃😭🧐🥙🍕🍔🚠🚆🛹🚋🚈🛴💫💨💝💌🕳️🚋🍕🥨🚡💦💕🥺🤔🐕🐎💞🏍️🏎️🍕💫😌🍕🤑💫💢💗☯️♌⛎😥😢♍😧😨🤨👉👈🈹

[karnikaavelumani]

#756 (comment)

@jaasonw can you take a look at this? For some reason the student-pack shortlink, which is supposed to link to https://acmcsuf.com/blog/806 link, is the only one that does not work. Do you know why?

[jaasonw]

is it not bcus they're duplicated?

[EthanThatOneKid]

is it not bcus they're duplicated?

No. We discovered a regression in the findURL function.

The current plan is to replace the code path that throws by passing a destinationOrigin to use when resolving a found shortlink missing a destination origin.

However, this means findURL finds a URL every time which strays from the expected behavior implied by the function name.

[EthanThatOneKid]

No. We discovered a regression in the findURL function.

The findURL regression has been followed up in #818.

[jaasonw]

https://docs.netlify.com/routing/redirects/ why not just do all these redirects directly in netlify

[EthanThatOneKid]

https://docs.netlify.com/routing/redirects/ why not just do all these redirects directly in netlify

We want the feature to work no matter which provider we deploy with.