-
Notifications
You must be signed in to change notification settings - Fork 70
Adoption Page Ideas #250
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adoption Page Ideas #250
Conversation
✅ Deploy Preview for awesome-golick-685c88 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
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. |
notmandatory
left a comment
There was a problem hiding this 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.
docs/adoption/all.md
Outdated
| <div class="tagline"> | ||
| <h3> | ||
| <a href="https://www.anchorwatch.com/" target="_blank">AnchorWatch</a> | ||
| <span class="foundation-member-badge">Foundation Member</span> |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
|
updated with the suggestion from Steve #250 (comment) and I'll take the LGTM #250 (review) as a go-ahead to merge now |
Sort members to top of adoption list, add a badge.
This pr will address/close bdk foundation issue 142