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

Issue 6: Add batch details in header #20

Merged
merged 5 commits into from
Sep 19, 2024

Conversation

arjanjohan
Copy link
Contributor

Description

Show if the connected wallet is member of the batch and checked in.
image
image

Additional Information

Related Issues

Closes #5

Your ENS/address: arjanjohan.eth

Copy link

vercel bot commented Sep 11, 2024

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

Name Status Preview Comments Updated (UTC)
batch9-buidlguidl-com-nextjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 18, 2024 7:09am

@arjanjohan arjanjohan changed the title Add batch details in header Issue 6: Add batch details in header Sep 11, 2024
@derrekcoleman
Copy link
Member

Your current design separates allow list from check in, but these two variables have a relationship with each other that makes it possible to simplify the UI.

  • What is the total set of possible states for someone who is or isn't logged in?
  • Are there any clever patterns we could use to simplify the way we display data under different conditions?

@derrekcoleman
Copy link
Member

Beautiful! Great collaboration, you two

@derrekcoleman derrekcoleman merged commit dcd9876 into BuidlGuidl:main Sep 19, 2024
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.

6. Show connected wallet info
4 participants