Skip to content

feat: add Gwill builder page#19

Merged
phipsae merged 9 commits intoBuidlGuidl:mainfrom
big14way:feature/gwill-builder-page
May 16, 2025
Merged

feat: add Gwill builder page#19
phipsae merged 9 commits intoBuidlGuidl:mainfrom
big14way:feature/gwill-builder-page

Conversation

@big14way
Copy link
Contributor

@big14way big14way commented May 9, 2025

Description

Added my personal builder page to the Batch 16 website. The page includes:

  • Personal introduction and bio
  • Profile image using GitHub avatar
  • Social media links (GitHub, Twitter, LinkedIn)
  • Favorite tech stack and current focus areas
  • Wallet address display using SE-2 Address component

Technical changes:

  • Created new page at packages/nextjs/app/builders/0x208B2660e5F62CDca21869b389c5aF9E7f0faE89/page.tsx
  • Added GitHub avatars domain to Next.js image configuration
  • Added react-icons dependency for social media icons
  • Implemented responsive design with Tailwind CSS

Screenshot of the page:
Screenshot 2025-05-09 at 10 18 18 AM

Additional Information

  • I have read the contributing docs
  • This is not a duplicate of any existing pull request

Related Issues

Closes #8 (Builder Personal Page)

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.

Copy link
Collaborator

@melanke melanke 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 the update. The PR description is perfect now. Just for your information, though — you didn’t have to close one PR and open another; you could have simply updated the first one.

Here's a new feedback:

Comment on lines +77 to +91
<div className="grid grid-cols-1 md:grid-cols-2 gap-4 w-full max-w-2xl">
<div className="stats shadow">
<div className="stat">
<div className="stat-title">Favorite Stack</div>
<div className="stat-value text-primary text-2xl">NextJS + Solidity</div>
</div>
</div>

<div className="stats shadow">
<div className="stat">
<div className="stat-title">Current Focus</div>
<div className="stat-value text-secondary text-2xl">DeFi + Startup Funding</div>
</div>
</div>
</div>
Copy link
Collaborator

Choose a reason for hiding this comment

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

These two divs are very subtle — they could use more contrast to be easily noticed. Especially the second one; it looks like it's fading away.

@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 16, 2025 10:33am

@big14way
Copy link
Contributor Author

i fixed it

@melanke
Copy link
Collaborator

melanke commented May 12, 2025

@big14way The second item is still hard to read on light-mode. Maybe it's because we had the light/dark theme problem I mentioned in Telegram.

Can you rebase your PR with the origin/main branch and check again?

@big14way
Copy link
Contributor Author

check it out

@melanke
Copy link
Collaborator

melanke commented May 12, 2025

Ok, it looks like you rebase correctly, but the problem remains.

@big14way
Copy link
Contributor Author

is the problem still there ?

@melanke
Copy link
Collaborator

melanke commented May 13, 2025

is the problem still there ?

@big14way Yes! Are you able to see the problem?

@big14way
Copy link
Contributor Author

check again

@melanke
Copy link
Collaborator

melanke commented May 14, 2025

image

@big14way
Copy link
Contributor Author

i think its better now

@melanke melanke requested a review from phipsae May 15, 2025 15:21
@phipsae phipsae merged commit a63d8f2 into BuidlGuidl:main May 16, 2025
3 checks passed
@phipsae
Copy link
Contributor

phipsae commented May 16, 2025

Looks great, let's merge, thx!

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.

b) Create your personal page

3 participants