تحديث أو إزالة رابط عرض توضيح Highlight.js

In highlighted languages in Settings, if you click on the link https://highlightjs.org/static/demo, it takes you to https://highlightjs.org/not-a-cdn which says…

Do not use us as a CDN. You should not be linking to any assets that are hosted on our website. We are getting million of garbage requests due to this misuse of our assets.

Perhaps the link should be https://highlightjs.org/demo

I believe the relevant string to be changed here (but will need changing in other languages).

5 إعجابات

I sent a PR to fix this, thank you for tracking down the code location: UX: fix link to Highlight JS demo by mr-c · Pull Request #29701 · discourse/discourse · GitHub

4 إعجابات

The only file you need to change is server.en.yml. The other ones are changed when translations from Crowdin are merged.

4 إعجابات

PR is now merged, thanks @biocrusoe, and thanks for reporting @yanokwa. Closing.

إعجابَين (2)