Add BuilderProfile by dmystical-coder · Pull Request #16 · BuidlGuidl/batch16.buidlguidl.com

dmystical-coder · 2025-05-08T22:41:54Z

Description

This PR adds my builder profile page to the batch site. I've created a new profile page with a little bio, avatar, address, skills and links to my socials.

Custom builder profile page with responsive design
Skills section highlighting blockchain and frontend technologies
Profile information with GitHub avatar and social links

Additional Information

I have read the contributing docs (if this is your first contribution)
This is not a duplicate of any existing pull request

Related Issues

This PR adds a new builder profile page to the platform as part of the Batch 16 contributions. Issue #8

Your ENS/address:
0x119d9A1ef0D16361284a9661727b363B04B5B0c8

vercel · 2025-05-08T22:41:58Z

@dmystical-coder is attempting to deploy a commit to the BuidlGuidl Team on Vercel.

A member of the Team first needs to authorize it.

dmystical-coder · 2025-05-08T22:43:31Z

I touched the next configuration file so I could properly load GitHub avatar images using remotePatterns.

melanke

Thank you for your personal page and for the perfect PR description :)

Here's my CR:

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/page.tsx

melanke · 2025-05-08T23:14:14Z

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/page.tsx

+  const frontendSkills = ["React", "TypeScript", "NextJS", "TailwindCSS"];
+
+  return (
+    <div className="min-h-screen bg-white text-black font-mono pt-4">


Your page looks good in light mode, but you might not have noticed the theme switcher at the bottom right corner. It would be great if your content adapted to the dark theme as well.

Now it only works in dark mode.

Edit: It seems to be a bigger problem, the dark variant is being ignored in the whole project. Let me investigate this.

Edit2: The dark/light theme problem is fixed in origin/main branch, can you rebase your branch? Thanks

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/page.tsx

melanke

Thanks for the update, here are a few more comments:

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/page.tsx

melanke · 2025-05-09T13:16:23Z

packages/nextjs/next-env.d.ts

@@ -1,5 +1,6 @@
 /// <reference types="next" />
 /// <reference types="next/image-types/global" />
+/// <reference types="next/navigation-types/compat/navigation" />


Can you please revert the change to next-env.d.ts?
It seems unrelated to the remotePatterns config in next.config.ts or any other change you made in your PR, and might have been added by mistake or by your local setup. This file is usually managed automatically by Next.js and shouldn’t be edited manually unless necessary.

Hi @melanke
I have tried to revert this change, but on committing the change, it reappears. Next.js is automatically regenerating the file during the build process.
Kindly advise.

It's funny, it happens the opposite when I open your branch here on my machine. It erases this line.

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/page.tsx

vercel · 2025-05-09T22:51:26Z

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 12, 2025 5:22pm

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/icons.tsx

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/page.tsx

melanke · 2025-05-12T11:06:50Z

packages/nextjs/next-env.d.ts

@@ -1,5 +1,6 @@
 /// <reference types="next" />
 /// <reference types="next/image-types/global" />
+/// <reference types="next/navigation-types/compat/navigation" />


It's funny, it happens the opposite when I open your branch here on my machine. It erases this line.

melanke · 2025-05-12T11:08:43Z

packages/nextjs/app/builders/0x119d9A1ef0D16361284a9661727b363B04B5B0c8/page.tsx

+  const frontendSkills = ["React", "TypeScript", "NextJS", "TailwindCSS"];
+
+  return (
+    <div className="min-h-screen bg-white text-black font-mono pt-4">


Now it only works in dark mode.

Edit: It seems to be a bigger problem, the dark variant is being ignored in the whole project. Let me investigate this.

Edit2: The dark/light theme problem is fixed in origin/main branch, can you rebase your branch? Thanks

Remove 'use client' directive from profile page to enable static rendering at build time. This improves performance by pre-rendering the page during compilation and reduces client-side JavaScript.

Replace repetitive social link elements with a mapping structure for better maintainability and reduced code duplication.

Remove manual changes to next-env.d.ts as it is an auto-generated file that should not be modified manually.

Move blockchainSkills, frontendSkills, socialLinks, and myAddress constants outside the component to prevent recreation on each render and improve performance.

Replace dynamic year calculation with static year 2025 in the footer message to accurately reflect the batch year.

Remove 'use client' directive from icon components since they are pure SVG elements that can be pre-rendered during build time, improving performance by reducing client-side JavaScript.

Remove hover:shadow-md class from footer message since it wasn't providing any visible effect and was adding unnecessary CSS to the bundle.

phipsae · 2025-05-12T20:57:13Z

@dmystical-coder, what a cool page! Really like the design, thank you so much for your effort! Let's merge :)

Thanks @melanke for the review!

melanke requested changes May 8, 2025

View reviewed changes

phipsae assigned dmystical-coder May 8, 2025

melanke requested changes May 9, 2025

View reviewed changes

vercel bot deployed to Preview May 9, 2025 22:53 View deployment

vercel bot deployed to Preview May 10, 2025 07:17 View deployment

MethelyDev mentioned this pull request May 12, 2025

Methely builder page #28

Closed

2 tasks

melanke requested changes May 12, 2025

View reviewed changes

vercel bot deployed to Preview May 12, 2025 12:52 View deployment

dmystical-coder added 13 commits May 12, 2025 18:17

Add BuilderProfile

51a9e93

Add icons as React components

9b7c252

Include navigation types for Next.js

1815451

Refactor BuilderProfile component

844f964

refactor: remove client directive for static rendering

8643f97

Remove 'use client' directive from profile page to enable static rendering at build time. This improves performance by pre-rendering the page during compilation and reduces client-side JavaScript.

refactor: ♻️ use mapping for social links

76c4fa1

Replace repetitive social link elements with a mapping structure for better maintainability and reduced code duplication.

revert: ⏪ restore next-env.d.ts to original state

30064fb

Remove manual changes to next-env.d.ts as it is an auto-generated file that should not be modified manually.

revert: ⏪ restore next-env.d.ts to original state

57559de

perf: ⚡ move static data outside component

a3c8674

Move blockchainSkills, frontendSkills, socialLinks, and myAddress constants outside the component to prevent recreation on each render and improve performance.

fix: use static year in footer

6656d85

Replace dynamic year calculation with static year 2025 in the footer message to accurately reflect the batch year.

perf: ⚡ remove client directive from icons

843da05

Remove 'use client' directive from icon components since they are pure SVG elements that can be pre-rendered during build time, improving performance by reducing client-side JavaScript.

chore: 🔥 remove unused hover effect

25b7fad

Remove hover:shadow-md class from footer message since it wasn't providing any visible effect and was adding unnecessary CSS to the bundle.

revert: ⏪ restore next-env.d.ts to original state

a3b5c81

dmystical-coder force-pushed the b-PersonalPage branch from 8e2c450 to a3b5c81 Compare May 12, 2025 17:20

vercel bot deployed to Preview May 12, 2025 17:22 View deployment

melanke approved these changes May 12, 2025

View reviewed changes

melanke requested a review from phipsae May 12, 2025 17:28

phipsae merged commit f057cf5 into BuidlGuidl:main May 12, 2025
3 checks passed

Conversation

dmystical-coder commented May 8, 2025

Description

Additional Information

Related Issues

Uh oh!

vercel bot commented May 8, 2025

Uh oh!

dmystical-coder commented May 8, 2025

Uh oh!

melanke left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

melanke May 8, 2025

Choose a reason for hiding this comment

Uh oh!

dmystical-coder May 9, 2025

Choose a reason for hiding this comment

Uh oh!

melanke May 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

melanke left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

melanke May 9, 2025

Choose a reason for hiding this comment

Uh oh!

dmystical-coder May 10, 2025

Choose a reason for hiding this comment

Uh oh!

melanke May 12, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

vercel bot commented May 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

melanke May 12, 2025

Choose a reason for hiding this comment

Uh oh!

melanke May 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

phipsae commented May 12, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

melanke May 12, 2025 •

edited

Loading

vercel bot commented May 9, 2025 •

edited

Loading

melanke May 12, 2025 •

edited

Loading