shields icon indicating copy to clipboard operation
shields copied to clipboard

Add [snapstore] version badge

Open jNullj opened this issue 1 year ago • 1 comments

Example badge render: image

Path: http://HOST/snapstore/v/:package

A few notes:

  1. Takes the latest value, might be a problem with mix of channels
  2. Currently does not use authentication - just proof of concept, I think we will have to add macaroon auth for this PR.
  3. I need to add tests.

Fixes badges/shields#9103

jNullj avatar Feb 17 '24 18:02 jNullj

Messages
:book: :sparkles: Thanks for your contribution to Shields, @jNullj!

Generated by :no_entry_sign: dangerJS against 29dd792cd42c823811b4614c75b0d1ffdaac8745

github-actions[bot] avatar Feb 17 '24 18:02 github-actions[bot]

I think we can get away without auth tbh. I think the macaroon auth is needed if you want to do stuff like publish snaps, for example. I'm not seeing anything that indicates the /info endpoint is rate limited without auth. Have you seen any docs indicating otherwise?

chris48s avatar Feb 19 '24 12:02 chris48s

🚀 Updated review app: https://pr-9976-badges-shields.fly.dev

github-actions[bot] avatar Mar 10 '24 10:03 github-actions[bot]

This latest iteration is solid. Nice work :+1:

chris48s avatar Mar 10 '24 10:03 chris48s

When will this be available? 🤔

jcs090218 avatar Mar 10 '24 10:03 jcs090218

I ran a deploy this evening It is live now https://shields.io/badges/snapcraft-version

chris48s avatar Mar 10 '24 19:03 chris48s

Thank you!

jcs090218 avatar Mar 10 '24 20:03 jcs090218