Skip to content

Conversation

@reez
Copy link
Collaborator

@reez reez commented Nov 17, 2025

Sort members to top of adoption list, add a badge.

This pr will address/close bdk foundation issue 142

Screenshot 2025-11-17 at 3 03 37 PM

@netlify
Copy link

netlify bot commented Nov 17, 2025

Deploy Preview for awesome-golick-685c88 ready!

Name Link
🔨 Latest commit 3d13a06
🔍 Latest deploy log https://app.netlify.com/projects/awesome-golick-685c88/deploys/691bc654e990f6000806c13a
😎 Deploy Preview https://deploy-preview-250--awesome-golick-685c88.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@reez
Copy link
Collaborator Author

reez commented Nov 17, 2025

I just put "foundation member" for both. They are silver members, didn't know how much text we wanted there, could also do "foundation member silver" or just "silver" or anything, let me know you're thoughts and I'll make it happen.

Copy link
Member

@notmandatory notmandatory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for doing this, I have one suggestion below. Otherwise LGTM.

<div class="tagline">
<h3>
<a href="https://www.anchorwatch.com/" target="_blank">AnchorWatch</a>
<span class="foundation-member-badge">Foundation Member</span>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make these clickable links to the foundation members page?: /foundation/members

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good call, wish I would have though of that. updated now

@reez
Copy link
Collaborator Author

reez commented Nov 18, 2025

updated with the suggestion from Steve #250 (comment)

and I'll take the LGTM #250 (review) as a go-ahead to merge now

@reez reez merged commit 7ef45a4 into bitcoindevkit:master Nov 18, 2025
5 checks passed
@reez reez deleted the badge branch November 18, 2025 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants