salesagility / SuiteDocs

SuiteCRM - Open source CRM for the world
https://docs.suitecrm.com/
Other
58 stars 284 forks source link

Avoid GitHub API failures (Release Notes) #483

Open pgorod opened 3 years ago

pgorod commented 3 years ago

Page with Issue

Our Release Notes pages use the GItHub API to fetch members' avatars. Currently, API failures might cause problems when building the site. And API failures are not that uncomon - not full GitHub outages, but a much simpler thing: exhausting API limits due to too many requests when building the site.

Underlying issue

This is related to a Hugo known issue, https://github.com/gohugoio/hugo/issues/6954

After a long time stalled, recently there was a fix for this, check it out.

What needs to be done here on SuiteDocs

This is the file that needs to get the improved error handling:

https://github.com/salesagility/SuiteDocs/blob/master/layouts/shortcodes/ghcontributors.html

samus-aran commented 2 years ago

Yup! This is what I now face.... Either jump ahead and provide the fall back now and work on the upgrade. I'm not sure if they can be done separately but an upgrade is always good.

serhiisamko091184 commented 3 months ago

Hello @pgorod,

I'd like to check if this issue is still valid?

Thanks in advance!

Regards, Serhii

pgorod commented 3 months ago

I guess it is. In general, the whole Docs stack should get some love (and upgrades). That would be Hugo, the Learn theme, asciidoc.