website icon indicating copy to clipboard operation
website copied to clipboard

Picture/Diagram missing on Documentation page Triggers and EventListeners

Open vovtz opened this issue 4 years ago • 11 comments

Expected Behavior

All images/diagrams in the documentation are present

Actual Behavior

The SVG image about the Trigger Flow does not exist (shows a broken image icon) on the Triggers and EventListeners page

Steps to Reproduce the Problem

  1. Open https://tekton.dev/docs/triggers/ in a browser

Additional Info

The SVG images URL is https://tekton.dev/docs/images/TriggerFlow.svg

vovtz avatar Feb 08 '22 18:02 vovtz

@tektoncd/triggers-maintainers fyi

AlanGreene avatar Feb 14 '22 11:02 AlanGreene

I looked a bit at this one, and the problem is that the image is not stored along with the docs, but in an images folder, which is a peer of the docs one. I suspect the easiest fix here would be to move the image around, possibly place the images folder under docs in the triggers repo would help already.

Alternatively we could add logic to the sync script to handle shared images, e.g.:

  • each project could specify one shared images folder
  • those images could be copied into the hugo static content folder, in a project/version specific subfolder
  • links to the image would be overwritten accordingly

This would quite a bit of engineering to be done for a feature that I'm not convinced we need.

afrittoli avatar Feb 14 '22 12:02 afrittoli

I'd be in favour of the easier approach here:

I suspect the easiest fix here would be to move the image around, possibly place the images folder under docs in the triggers repo would help already.

AlanGreene avatar Feb 14 '22 12:02 AlanGreene

Sure, I would move the images folder under the docs folder. It's fast and simple.

khrm avatar Feb 14 '22 13:02 khrm

Thanks for the fast resolution. When will this result in a change of the effected website page (https://tekton.dev/docs/triggers/)?

vovtz avatar Feb 14 '22 18:02 vovtz

@afrittoli I think placing images folder inside triggers docs folders didn't solve this. Or is the website yet to be rebuilt?

khrm avatar Feb 15 '22 06:02 khrm

Thanks @khrm I need to update the website config to include the new folder. Once I've done that it should on the "main" branch docs. If you want to fix it on the docs associated to a release you'll need to backport the fix to the branch

afrittoli avatar Feb 15 '22 07:02 afrittoli

This came up during the Docs WG today when reviewing potential issues for new contributors.

@afrittoli I see you have a PR open for this already: https://github.com/tektoncd/website/pull/351

Does it still require more work? If so, do you think you'll have bandwidth to work on it or does it need a new owner?

AlanGreene avatar Apr 26 '22 17:04 AlanGreene

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale with a justification. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close with a justification. If this issue should be exempted, mark the issue as frozen with /lifecycle frozen with a justification.

/lifecycle stale

Send feedback to tektoncd/plumbing.

tekton-robot avatar Jul 25 '22 17:07 tekton-robot

/remove-lifecycle stale

Still looks like:

image

vovtz avatar Jul 29 '22 08:07 vovtz

The <img/> tag references https://tekton.dev/docs/triggers/images/TriggerFlow.svg (indirectly, by specifying path ./images/TriggerFlow.svg), which doesn’t exist.

vovtz avatar Jul 29 '22 08:07 vovtz

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale with a justification. Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with /close with a justification. If this issue should be exempted, mark the issue as frozen with /lifecycle frozen with a justification.

/lifecycle stale

Send feedback to tektoncd/plumbing.

tekton-robot avatar Oct 27 '22 09:10 tekton-robot

/remove-lifecycle stale

AlanGreene avatar Oct 27 '22 09:10 AlanGreene