Skip to content
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

5. Get the check-in count from the BatchRegistry contract #6

Closed
phipsae opened this issue Oct 29, 2024 · 9 comments · Fixed by #35
Closed

5. Get the check-in count from the BatchRegistry contract #6

phipsae opened this issue Oct 29, 2024 · 9 comments · Fixed by #35
Labels

Comments

@phipsae
Copy link
Collaborator

phipsae commented Oct 29, 2024

Goal

There is a "To Be implemented" string in packages/nextjs/app/page.tsx that should show the number of builders that have checked in to the contract.

We want to show the real count (reading from the BatchRegistry contract)

For more info about the contract: #10

@Vedant-05
Copy link

Is Someone working on this one ?

@Vedant-05
Copy link

This is only to get the Checkin Count .
So maybe it should be added to the main page tweaks only, where we are listing all the team members that checkin ?

@phipsae
Copy link
Collaborator Author

phipsae commented Nov 29, 2024

That are all three separate issues, checkedin counter, list all the team members (@gboigwe is working on that) and tweak main page (there is a discussion ongoing here #7).

@Vedant-05
Copy link

ok

@Vedant-05
Copy link

so this checkin counter is an open issue right ?

@phipsae
Copy link
Collaborator Author

phipsae commented Nov 29, 2024

Since you worked on the logo, perhaps let someone else take on the task over the weekend.
If it’s still open after that, you can go ahead and tackle it.

But feel free to work on #4, align with the others.

@Vedant-05
Copy link

Yes definately!

@Asante-ntiro
Copy link
Contributor

Can I take this on? I see it's still open.

@phipsae
Copy link
Collaborator Author

phipsae commented Dec 10, 2024

Yes, go for it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants