backdrop-ops / docs.backdropcms.org

Website for displaying Backdrop CMS documentation and API source code.
https://docs.backdropcms.org/
6 stars 6 forks source link

Missing figure on "The Figure-8 approach" #210

Closed kswan closed 1 year ago

kswan commented 1 year ago

There is a broken image after the first paragraph on https://docs.backdropcms.org/documentation/the-figure-8-approach

image

bugfolder commented 1 year ago

@kswan, here's how it looks on Safari on Mac (also Chrome and Firefox):

figure

What OS/browser are you using?

kswan commented 1 year ago

I tested on Edge, Chrome and Firefox and none show the image, but now I believe our corporate proxy is blocking the image.

I noticed that the image source is https://i.postimg.cc/HWZtXXMT/config-deploy-strategy.png

Is it normal that d.b.o uses postimg.cc for image hosting?

Since this is clearly a problem on my end, I'll close the issue.

bugfolder commented 1 year ago

@kswan, I don't think we should be using an offsite hosting service for this image (or any image on our docs site), so I'm reopening this issue. Pinging @jenlampton, who probably knows the history and rationale of this page.

kswan commented 1 year ago

OK, thank you. I was digging through other pages trying to find whether images were hosted on-site or off-site. There aren't many images on d.b.o, but when I finally found a page with images (https://docs.backdropcms.org/documentation/user-guide) they were on-site.

jenlampton commented 1 year ago

I have added https://docs.backdropcms.org/file/11, and updated the src for the image to point to the local file.

For history: this page was migrated from the b.org site, where there once was an image field on documentation nodes (back from before we had inline image support, and before the inclusion of file_entity). When we pulled across the content, we missed some of the images.

Thank you for spotting the problem!