Skip to content

feat: implement checked-in builders counter display#22

Merged
phipsae merged 2 commits intoBuidlGuidl:mainfrom
big14way:feature/checked-in-counter
May 14, 2025
Merged

feat: implement checked-in builders counter display#22
phipsae merged 2 commits intoBuidlGuidl:mainfrom
big14way:feature/checked-in-counter

Conversation

@big14way
Copy link
Contributor

@big14way big14way commented May 9, 2025

Description

Implemented the checked-in builders counter display on the homepage. This replaces the "To Be implemented" placeholder with the actual count of builders who have checked in to the BatchRegistry contract.

Changes made:

  • Added useScaffoldReadContract hook to read the checkedInCounter from BatchRegistry contract
  • Updated the display to show the actual count with a fallback to "0"
  • The counter updates automatically when new builders check in

Screenshot:
Screenshot 2025-05-09 at 11 25 59 AM

Additional Information

Related Issues

closes (#5 )

This implements part of the batch16.buidlguidl.com setup requirements.

Your ENS/address: 0x208B2660e5F62CDca21869b389c5aF9E7f0faE89

@vercel
Copy link

vercel bot commented May 9, 2025

@big14way is attempting to deploy a commit to the BuidlGuidl Team on Vercel.

A member of the Team first needs to authorize it.

@melanke melanke requested a review from phipsae May 9, 2025 14:08
@vercel
Copy link

vercel bot commented May 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
batch16.buidlguidl.com ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 14, 2025 3:56am

Copy link
Contributor

@phipsae phipsae left a comment

Choose a reason for hiding this comment

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

Looks great, thank you!

Just a tiny comment below.

@phipsae
Copy link
Contributor

phipsae commented May 14, 2025

Thanks, like the change with the spinner! Let's merge.

@phipsae phipsae merged commit 513c718 into BuidlGuidl:main May 14, 2025
3 checks passed
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.

3 participants