search

appwrite / website

The Appwrite website, docs and blog 🏠
https://appwrite.io
195 stars 187 forks source link

Fix: Broken `XML` due to Title and Urls in `RSS` #1035

Closed ItzNotABug closed 3 months ago

ItzNotABug commented 3 months ago

What does this PR do?

An & and a (space) broke the xml format. This should fix the issue.

Test Plan

Manual check via https://validator.w3.org/feed.

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

Yes.

ItzNotABug commented 3 months ago

@TorstenDittmann should we keep description if null?

TorstenDittmann commented 3 months ago

@TorstenDittmann should we keep description if null?

There should always be a description 👍🏻

can we check and use https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/encodeURI for the urls?

ItzNotABug commented 3 months ago

@TorstenDittmann should we keep description if null?

There should always be a description 👍🏻

can we check and use https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/encodeURI for the urls?

I am actually using encodeURI, just in a function. Or am I missing something here 🤔?