FEATURE: Add a site setting to allow emojis to come from an external URL (PR #12180)

GitHub

I don’t think we need 2 settings here, it just grows our json paylod for no huge benefit. A single external_emoji_url can work. If it is blank then just assume not using external.

Should this go into a setupData instead of a client site setting, so that it could potentially be the same as S3CDN / SITE_CDN on self-hosted sites that have that?

@riking I think we get that for free anyway, cause we are using getURL etc internally.

Test failures look legit @xfalcox

127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/joy.png?v=9 HTTP/1.1" 404 - 0.0121
127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/relaxed.png?v=9 HTTP/1.1" 404 - 0.0155
127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/wink.png?v=9 HTTP/1.1" 404 - 0.0264
127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/neutral_face.png?v=9 HTTP/1.1" 404 - 0.01

I am guessing something is not set right somewhere.

Test failures look legit @xfalcox

127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/joy.png?v=9 HTTP/1.1" 404 - 0.0121
127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/relaxed.png?v=9 HTTP/1.1" 404 - 0.0155
127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/wink.png?v=9 HTTP/1.1" 404 - 0.0264
127.0.0.1 - - [23/Feb/2021:22:42:00 +0000] "GET /undefined/emoji_one/neutral_face.png?v=9 HTTP/1.1" 404 - 0.01

I am guessing something is not set right somewhere.

I was aware and chasing this for some long hours :sleepy:

Add a test that download_remote_images will ignore these?

Add a test that download_remote_images will ignore these?

Added on 807471d