sensu / catalog

Monitoring as code for Sensu Go. "There's a template for that!"
8 stars 4 forks source link

Link to Bonsai page instead of asset reference for Haproxy #338

Open asachs01 opened 2 years ago

asachs01 commented 2 years ago
CleanShot 2022-09-21 at 10 00 17

When clicking the asset link here ^ it takes me to a reference on assets, which is kind of useless. Instead, the asset link should link to the actual bonsai asset that's being used.

hillaryfraley commented 2 years ago

~This one is also fixed in https://github.com/sensu/catalog/pull/284 (awaiting review/publish)~

edit: sorry my mistake -- the asset page is the correct link there (similarly, the check link is to the check reference). These links are intended to explain the resource type. The link to the bonsai asset page is further down in the readme, when we list the plugins that the integration uses

asachs01 commented 2 years ago

Oh! SWEEEEET!

On Wed, Sep 21, 2022 at 10:38 Hillary Fraley @.***> wrote:

This one is also fixed in #284 https://github.com/sensu/catalog/pull/284 (awaiting review/publish)

— Reply to this email directly, view it on GitHub https://github.com/sensu/catalog/issues/338#issuecomment-1253807725, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG3MQYH6AVIEQ2LQYPAXKDV7MMWTANCNFSM6AAAAAAQSDNUTI . You are receiving this because you authored the thread.Message ID: @.***>

-- Sent from Gmail Mobile