Skip to content

Refactor code structure and implement layout components#2

Merged
danielmarv merged 4 commits intonext-js-migrationfrom
daniel-dev-rebuild
Mar 11, 2026
Merged

Refactor code structure and implement layout components#2
danielmarv merged 4 commits intonext-js-migrationfrom
daniel-dev-rebuild

Conversation

@danielmarv
Copy link
Copy Markdown
Member

Description

This pull request introduces a major redesign and foundational setup for the Hiero website using Next.js. It replaces the default Next.js template with a custom layout, adds a new header, footer, and responsive navigation menu, and introduces supporting data and configuration files. The changes also lay the groundwork for future homepage sections and content migration from the previous Hugo-based site.

Site Layout & UI Overhaul:

  • Replaced the default Next.js template with a custom homepage (page.tsx) that introduces Hiero branding, a migration notice, and new call-to-action buttons. The new layout uses a modern, branded color scheme and typography.
  • Added a persistent Header and Footer component, integrated into the main layout (layout.tsx), providing consistent navigation and site information across all pages. [1] [2] [3] [4]
  • Implemented a responsive Menu component for navigation, supporting both mobile and desktop experiences, with links to key resources and external sites.

Styling & Fonts:

  • Switched the primary font to Space Grotesk and the monospace font to IBM Plex Mono for a more distinctive and modern look. Updated CSS variables and font weights accordingly.

Data & Configuration:

  • Added repository_stats.json and technical_steering_committee.json to provide structured data for repository metrics and committee member profiles, supporting future dynamic content. [1] [2]
  • Updated next.config.ts to allow loading images from GitHub avatars, enabling richer profile displays.

Dependencies:

  • Introduced new dependencies (date-fns, gray-matter, remark, remark-html, swiper, toml) to support content parsing, formatting, and future interactive features.

Supporting Components:

  • Added a reusable Container component for consistent page padding and alignment.

These changes establish a strong foundation for the Hiero website's migration and future development, focusing on branding, usability, and scalability.

Changes Made

  • Added...
  • Modified...
  • Fixed...

Related Issues

Screenshots (if applicable)

Checklist

  • Tests added/updated
  • Documentation updated
  • Linting passes
  • Branch up-to-date with main

Deployment Notes

…bility

Signed-off-by: Daniel Ntege <danientege785@gmail.com>
…layout

Signed-off-by: Daniel Ntege <danientege785@gmail.com>
Signed-off-by: Daniel Ntege <danientege785@gmail.com>
Copilot AI review requested due to automatic review settings March 11, 2026 12:33
@danielmarv danielmarv self-assigned this Mar 11, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR replaces the default Next.js starter UI with a Hiero-branded layout foundation (header/footer/navigation), sets up Tailwind v4 theming in globals.css, and adds initial structured data + public assets to support the ongoing Hugo → Next.js migration.

Changes:

  • Introduces persistent layout components (Header, Footer, Menu, Container) and wires them into app/layout.tsx.
  • Replaces the starter homepage with a migration notice + basic CTAs and applies new global Tailwind theme/styles.
  • Adds supporting data JSON files and a large set of static brand/illustration assets; updates Next image remote patterns.

Reviewed changes

Copilot reviewed 11 out of 108 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
.gitignore Adjusts ignore rules around /public to keep nextjs/public tracked.
nextjs/app/globals.css Adds Tailwind v4 @theme tokens and extensive global styling (nav/container/prose/swiper/etc.).
nextjs/app/layout.tsx Switches fonts to Space Grotesk / IBM Plex Mono; mounts Header/Footer; updates metadata/icons.
nextjs/app/page.tsx Replaces starter page with branded migration notice + CTAs.
nextjs/components/Container.tsx Adds reusable page container wrapper.
nextjs/components/Footer.tsx Adds site footer with LF Projects link.
nextjs/components/Header.tsx Adds fixed header with logo + menu.
nextjs/components/Menu.tsx Adds responsive navigation (mobile overlay + desktop links).
nextjs/data/repository_stats.json Adds repo stats seed data for future dynamic sections.
nextjs/data/technical_steering_committee.json Adds TSC member seed data for future dynamic sections.
nextjs/next.config.ts Allows next/image to load GitHub avatar images via remotePatterns.
nextjs/package.json Adds content/date/swiper parsing dependencies (date-fns, gray-matter, remark, swiper, toml).
nextjs/public/file.svg Adds static icon asset.
nextjs/public/globe.svg Adds static icon asset.
nextjs/public/next.svg Adds static icon asset.
nextjs/public/vercel.svg Adds static icon asset.
nextjs/public/window.svg Adds static icon asset.
nextjs/public/images/Hiero-fair-cover.png Adds static brand/image asset.
nextjs/public/images/Hiero-Favicon.svg Adds favicon asset.
nextjs/public/images/Hiero-Icon-Decentralized.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Fair.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Fast.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Github.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Heart.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Leaderless.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-ModalClose.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Nav-Menu.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Open.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-OpenSource.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Quote-Left.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Quote-Right.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Secure.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Timeline-1.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Timeline-2.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Timeline-3.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Timeline-4.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Timeline-5.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-Timeline-6.svg Adds static icon asset.
nextjs/public/images/Hiero-Icon-wLogo-white-text.svg Adds static icon/wordmark asset.
nextjs/public/images/Hiero-Icon-wLogo.svg Adds static icon/wordmark asset.
nextjs/public/images/Hiero-Icon.svg Adds static icon asset.
nextjs/public/images/Hiero-Logo-Calaxy.png Adds static logo asset.
nextjs/public/images/Hiero-Logo-DiamondStandard.svg Adds static logo asset.
nextjs/public/images/Hiero-Logo-Hedera.png Adds static logo asset.
nextjs/public/images/Hiero-Logo-Hgraph.png Adds static logo asset.
nextjs/public/images/Hiero-Logo-IBM.png Adds static logo asset.
nextjs/public/images/Hiero-Logo-HashPack.png Adds static logo asset.
nextjs/public/images/Hiero-Logo-Hashgraph.png Adds static logo asset.
nextjs/public/images/Hiero-Logo-Outline.svg Adds static logo asset.
nextjs/public/images/Hiero-Logo-Sentx.png Adds static logo asset.
nextjs/public/images/Hiero-Logo-ioBuilders.png Adds static logo asset.
nextjs/public/images/members/Mounil2005.png Adds member image asset.
nextjs/public/images/members/exploreriii.png Adds member image asset.
nextjs/public/images/profile-hiero.png Adds static profile image asset.
Files not reviewed (1)
  • nextjs/pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

href: "https://github.com/issues?q=is%3Aopen%20is%3Aissue%20org%3Ahiero-ledger%20archived%3Afalse%20no%3Aassignee%20(label%3A%22good%20first%20issue%22%20OR%20label%3A%22skill%3A%20good%20first%20issue%22)%20(repo%3Ahiero-ledger%2Fhiero-sdk-cpp%20OR%20repo%3Ahiero-ledger%2Fhiero-sdk-swift%20OR%20repo%3Ahiero-ledger%2Fhiero-sdk-python%20OR%20repo%3Ahiero-ledger%2Fhiero-sdk-js%20OR%20repo%3Ahiero-ledger%2Fhiero-website)",
external: true,
},
{ name: "Connect", href: "/#connect" },
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

The menu includes a /#connect anchor link, but there is no element with id="connect" in the current app/page.tsx, so the link won’t navigate anywhere. Consider removing/hiding this item until the section exists, or add the corresponding anchor target on the homepage.

Suggested change
{ name: "Connect", href: "/#connect" },

Copilot uses AI. Check for mistakes.
Comment thread nextjs/components/Menu.tsx Outdated
<nav
id="navigation"
className={`${isOpen ? "active-navigation" : "hidden"} fixed inset-0 z-50 flex-col items-center justify-start overflow-y-auto bg-black px-6 pt-24 pb-10 text-white sm:relative sm:inset-auto sm:z-auto sm:h-auto sm:w-9/12 sm:max-w-xl sm:flex sm:flex-row sm:items-center sm:justify-end sm:overflow-visible sm:bg-transparent sm:px-0 sm:pt-0 sm:pb-0 sm:text-inherit`}
aria-hidden={!isOpen && pathname !== undefined}
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

aria-hidden is set to true whenever isOpen is false, which will also apply on desktop (where the nav is visible via sm:flex). This removes the navigation from the accessibility tree for screen readers even though it’s on screen. Consider removing aria-hidden entirely (since hidden already removes it on mobile) or only applying it in the mobile-overlay state.

Suggested change
aria-hidden={!isOpen && pathname !== undefined}

Copilot uses AI. Check for mistakes.
Comment thread nextjs/app/page.tsx
Comment on lines 21 to +26
<a
href="https://nextjs.org/learn?utm_source=create-next-app&utm_medium=appdir-template-tw&utm_campaign=create-next-app"
className="font-medium text-zinc-950 dark:text-zinc-50"
href="/blog"
className="w-full rounded-full border border-white px-6 py-3 text-center font-medium text-white no-underline transition-colors hover:bg-white hover:text-charcoal sm:w-auto sm:min-w-[12rem]"
>
Learning
</a>{" "}
center.
</p>
</div>
<div className="flex flex-col gap-4 text-base font-medium sm:flex-row">
<a
className="flex h-12 w-full items-center justify-center gap-2 rounded-full bg-foreground px-5 text-background transition-colors hover:bg-[#383838] dark:hover:bg-[#ccc] md:w-[158px]"
href="https://vercel.com/new?utm_source=create-next-app&utm_medium=appdir-template-tw&utm_campaign=create-next-app"
target="_blank"
rel="noopener noreferrer"
>
<Image
className="dark:invert"
src="/vercel.svg"
alt="Vercel logomark"
width={16}
height={16}
/>
Deploy Now
</a>
<a
className="flex h-12 w-full items-center justify-center rounded-full border border-solid border-black/[.08] px-5 transition-colors hover:border-transparent hover:bg-black/[.04] dark:border-white/[.145] dark:hover:bg-[#1a1a1a] md:w-[158px]"
href="https://nextjs.org/docs?utm_source=create-next-app&utm_medium=appdir-template-tw&utm_campaign=create-next-app"
target="_blank"
rel="noopener noreferrer"
>
Documentation
</a>
Blog (Next.js)
</a>
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

The /blog call-to-action links to a route that doesn’t exist in this Next.js app (nextjs/app has no blog segment), so it will 404 at runtime. Either add a nextjs/app/blog/page.tsx (and supporting routes) or change this link to an existing destination (e.g., the legacy Hugo blog URL) until migration is complete.

Copilot uses AI. Check for mistakes.
external: true,
},
{ name: "Connect", href: "/#connect" },
{ name: "Blog", href: "/blog" },
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

The navigation includes a /blog link, but there is no nextjs/app/blog route in this PR, so this will render as a broken link (404). Add the blog route(s) or point to the current blog location until the Next.js blog is implemented.

Suggested change
{ name: "Blog", href: "/blog" },
{ name: "Blog", href: "/#blog" },

Copilot uses AI. Check for mistakes.
Comment thread nextjs/app/layout.tsx
>
{children}
<body className={`${spaceGrotesk.className} ${spaceGrotesk.variable} ${ibmPlexMono.variable}`}>
<header className="relative h-22.5">
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

h-22.5 isn’t a default Tailwind spacing utility and there’s no spacing scale defined in @theme to support it, so this class likely won’t generate and the wrapper won’t reserve space for the fixed header. Use an arbitrary value (e.g. h-[90px]) or a defined spacing token that matches the fixed header height.

Suggested change
<header className="relative h-22.5">
<header className="relative h-[90px]">

Copilot uses AI. Check for mistakes.
Comment thread nextjs/app/globals.css
content: '';
position: absolute;
bottom: 2rem;
left: 25%;
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

The mobile active underline is positioned with left: 25% plus transform: translateX(-50%), which mathematically places the underline on the left half of the link rather than centered. If the intent is to center a 50%-width underline, left should be 50% (or remove the translate and set left: 25% without translating).

Suggested change
left: 25%;
left: 50%;

Copilot uses AI. Check for mistakes.
Comment on lines +6 to +48
"photo": "richard.jpg",
"bio": "Richard is Vice President of Engineering at Hashgraph. Before the Hedera ecosystem, he was a Chief Client Architect for the Java platform at Oracle, and an architect for Oracle’s IoT applications. With 20+ years of experience building mission-critical platforms and applications used by millions of people worldwide, his passion is building incredible, scalable, high performance systems. He loves backpacking, camping, and watching his kids grow."
},
{
"firstName": "Dr. Leemon",
"lastName":"Baird",
"gitHubAccount": "https://github.com/lbaird",
"photo": "leemon.jpg",
"bio": "Leemon is the Co-founder and Chief Scientist at Hashgraph. The inventor of the organization’s namesake, the distributed hashgraph consensus algorithm, Leemon co-founded the Hedera network with Mance Harmon. He has two decades of technology and start-up experience, and served as Professor of Computer Science at the US Air Force Academy, as well as holding senior scientist roles in several labs. Leemon received his PhD in Computer Science from Carnegie Mellon University and has multiple patents and publications in peer-reviewed journals. He has spoken at conferences in computer security, machine learning and mathematics."
},
{
"firstName": "Hendrik",
"lastName":"Ebbers",
"gitHubAccount": "https://github.com/hendrikebbers",
"photo": "hendrik.jpg",
"bio": "Hendrik Ebbers is a member of the Board of Directors of the Eclipse Foundation. He is leading several open-source projects as a member of working groups and committees like Eclipse Adoptium or JakartaEE and actively contributing to even more projects. Hendrik has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Stoyan",
"lastName":"Panayotov",
"gitHubAccount": "https://github.com/stoqnkpL",
"photo": "stoyan.jpg",
"bio": "Stoyan Panayotov is a software architect at Limechain with several years of experience in the blockchain industry consulting various clients - decentralized networks, protocols and dApps. Stoyan has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Alex",
"lastName":"Popowycz",
"gitHubAccount": "https://github.com/popowycz",
"photo": "alex.jpg",
"bio": "Alex is the Chief Information Officer of Hedera and is charged with the technical operations of the network in close partnership with Hedera's member operators, ensuring the sustainable evolution of capabilities to the network while maintaining its core attributes of speed, fairness, and security. Prior to joining Hedera, Alex was the CIO at Health First, a multi-hospital integrated health system. He previously held several technology leadership positions at Fidelity Investments, with primary responsibility for cybersecurity and standards."
},
{
"firstName": "Georgi",
"lastName":"Lazarov",
"gitHubAccount": "https://github.com/georgi-l95",
"photo": "georgi.jpeg",
"bio": "Georgi Lazarov is a technical lead at LimeChain with more than 5 years of experience in the blockchain industry. He is a core committer to the tooling around Hiero, like Local Node and JSON-RPC Relay, as well as a contributor to the Block Node and other core components of the Hiero ecosystem."
},
{
"firstName": "Milan",
"lastName":"Wiercx van Rhijn",
"gitHubAccount": "https://github.com/MilanWR",
"photo": "milan.jpeg",
Copy link

Copilot AI Mar 11, 2026

Choose a reason for hiding this comment

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

The TSC data references local photo filenames (e.g., richard.jpg, leemon.jpg, etc.), but those files aren’t present under nextjs/public in this PR. If the UI uses this data, it will produce broken images; either add the referenced images, correct the paths, or switch photo to a URL (e.g., GitHub avatar) and adjust the rendering accordingly.

Suggested change
"photo": "richard.jpg",
"bio": "Richard is Vice President of Engineering at Hashgraph. Before the Hedera ecosystem, he was a Chief Client Architect for the Java platform at Oracle, and an architect for Oracle’s IoT applications. With 20+ years of experience building mission-critical platforms and applications used by millions of people worldwide, his passion is building incredible, scalable, high performance systems. He loves backpacking, camping, and watching his kids grow."
},
{
"firstName": "Dr. Leemon",
"lastName":"Baird",
"gitHubAccount": "https://github.com/lbaird",
"photo": "leemon.jpg",
"bio": "Leemon is the Co-founder and Chief Scientist at Hashgraph. The inventor of the organization’s namesake, the distributed hashgraph consensus algorithm, Leemon co-founded the Hedera network with Mance Harmon. He has two decades of technology and start-up experience, and served as Professor of Computer Science at the US Air Force Academy, as well as holding senior scientist roles in several labs. Leemon received his PhD in Computer Science from Carnegie Mellon University and has multiple patents and publications in peer-reviewed journals. He has spoken at conferences in computer security, machine learning and mathematics."
},
{
"firstName": "Hendrik",
"lastName":"Ebbers",
"gitHubAccount": "https://github.com/hendrikebbers",
"photo": "hendrik.jpg",
"bio": "Hendrik Ebbers is a member of the Board of Directors of the Eclipse Foundation. He is leading several open-source projects as a member of working groups and committees like Eclipse Adoptium or JakartaEE and actively contributing to even more projects. Hendrik has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Stoyan",
"lastName":"Panayotov",
"gitHubAccount": "https://github.com/stoqnkpL",
"photo": "stoyan.jpg",
"bio": "Stoyan Panayotov is a software architect at Limechain with several years of experience in the blockchain industry consulting various clients - decentralized networks, protocols and dApps. Stoyan has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Alex",
"lastName":"Popowycz",
"gitHubAccount": "https://github.com/popowycz",
"photo": "alex.jpg",
"bio": "Alex is the Chief Information Officer of Hedera and is charged with the technical operations of the network in close partnership with Hedera's member operators, ensuring the sustainable evolution of capabilities to the network while maintaining its core attributes of speed, fairness, and security. Prior to joining Hedera, Alex was the CIO at Health First, a multi-hospital integrated health system. He previously held several technology leadership positions at Fidelity Investments, with primary responsibility for cybersecurity and standards."
},
{
"firstName": "Georgi",
"lastName":"Lazarov",
"gitHubAccount": "https://github.com/georgi-l95",
"photo": "georgi.jpeg",
"bio": "Georgi Lazarov is a technical lead at LimeChain with more than 5 years of experience in the blockchain industry. He is a core committer to the tooling around Hiero, like Local Node and JSON-RPC Relay, as well as a contributor to the Block Node and other core components of the Hiero ecosystem."
},
{
"firstName": "Milan",
"lastName":"Wiercx van Rhijn",
"gitHubAccount": "https://github.com/MilanWR",
"photo": "milan.jpeg",
"photo": "https://avatars.githubusercontent.com/rbair23",
"bio": "Richard is Vice President of Engineering at Hashgraph. Before the Hedera ecosystem, he was a Chief Client Architect for the Java platform at Oracle, and an architect for Oracle’s IoT applications. With 20+ years of experience building mission-critical platforms and applications used by millions of people worldwide, his passion is building incredible, scalable, high performance systems. He loves backpacking, camping, and watching his kids grow."
},
{
"firstName": "Dr. Leemon",
"lastName":"Baird",
"gitHubAccount": "https://github.com/lbaird",
"photo": "https://avatars.githubusercontent.com/lbaird",
"bio": "Leemon is the Co-founder and Chief Scientist at Hashgraph. The inventor of the organization’s namesake, the distributed hashgraph consensus algorithm, Leemon co-founded the Hedera network with Mance Harmon. He has two decades of technology and start-up experience, and served as Professor of Computer Science at the US Air Force Academy, as well as holding senior scientist roles in several labs. Leemon received his PhD in Computer Science from Carnegie Mellon University and has multiple patents and publications in peer-reviewed journals. He has spoken at conferences in computer security, machine learning and mathematics."
},
{
"firstName": "Hendrik",
"lastName":"Ebbers",
"gitHubAccount": "https://github.com/hendrikebbers",
"photo": "https://avatars.githubusercontent.com/hendrikebbers",
"bio": "Hendrik Ebbers is a member of the Board of Directors of the Eclipse Foundation. He is leading several open-source projects as a member of working groups and committees like Eclipse Adoptium or JakartaEE and actively contributing to even more projects. Hendrik has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Stoyan",
"lastName":"Panayotov",
"gitHubAccount": "https://github.com/stoqnkpL",
"photo": "https://avatars.githubusercontent.com/stoqnkpL",
"bio": "Stoyan Panayotov is a software architect at Limechain with several years of experience in the blockchain industry consulting various clients - decentralized networks, protocols and dApps. Stoyan has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Alex",
"lastName":"Popowycz",
"gitHubAccount": "https://github.com/popowycz",
"photo": "https://avatars.githubusercontent.com/popowycz",
"bio": "Alex is the Chief Information Officer of Hedera and is charged with the technical operations of the network in close partnership with Hedera's member operators, ensuring the sustainable evolution of capabilities to the network while maintaining its core attributes of speed, fairness, and security. Prior to joining Hedera, Alex was the CIO at Health First, a multi-hospital integrated health system. He previously held several technology leadership positions at Fidelity Investments, with primary responsibility for cybersecurity and standards."
},
{
"firstName": "Georgi",
"lastName":"Lazarov",
"gitHubAccount": "https://github.com/georgi-l95",
"photo": "https://avatars.githubusercontent.com/georgi-l95",
"bio": "Georgi Lazarov is a technical lead at LimeChain with more than 5 years of experience in the blockchain industry. He is a core committer to the tooling around Hiero, like Local Node and JSON-RPC Relay, as well as a contributor to the Block Node and other core components of the Hiero ecosystem."
},
{
"firstName": "Milan",
"lastName":"Wiercx van Rhijn",
"gitHubAccount": "https://github.com/MilanWR",
"photo": "https://avatars.githubusercontent.com/MilanWR",

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What Copilot says is wrong and should not be done like that. Pictures are in the public/images/tsc folder

Signed-off-by: Daniel Ntege <danientege785@gmail.com>
Copy link
Copy Markdown
Member

@hendrikebbers hendrikebbers left a comment

Choose a reason for hiding this comment

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

I'm not really able to review TS / Next.JS code since that is not my area of expertise. I checked https://2.hiero.playground.open-elements.cloud and can see a base page without any content like TSC members or so. If that is the goal of this PR than I'm fine :)

Comment thread nextjs/public/next.svg
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I believe we can delete the NEXT JS images

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

will do that

@@ -0,0 +1,62 @@
{
"hiero-consensus-node": {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

How is that data updated? Is there an GitHub action in the original webpage for it?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

going to look into it

Comment on lines +6 to +48
"photo": "richard.jpg",
"bio": "Richard is Vice President of Engineering at Hashgraph. Before the Hedera ecosystem, he was a Chief Client Architect for the Java platform at Oracle, and an architect for Oracle’s IoT applications. With 20+ years of experience building mission-critical platforms and applications used by millions of people worldwide, his passion is building incredible, scalable, high performance systems. He loves backpacking, camping, and watching his kids grow."
},
{
"firstName": "Dr. Leemon",
"lastName":"Baird",
"gitHubAccount": "https://github.com/lbaird",
"photo": "leemon.jpg",
"bio": "Leemon is the Co-founder and Chief Scientist at Hashgraph. The inventor of the organization’s namesake, the distributed hashgraph consensus algorithm, Leemon co-founded the Hedera network with Mance Harmon. He has two decades of technology and start-up experience, and served as Professor of Computer Science at the US Air Force Academy, as well as holding senior scientist roles in several labs. Leemon received his PhD in Computer Science from Carnegie Mellon University and has multiple patents and publications in peer-reviewed journals. He has spoken at conferences in computer security, machine learning and mathematics."
},
{
"firstName": "Hendrik",
"lastName":"Ebbers",
"gitHubAccount": "https://github.com/hendrikebbers",
"photo": "hendrik.jpg",
"bio": "Hendrik Ebbers is a member of the Board of Directors of the Eclipse Foundation. He is leading several open-source projects as a member of working groups and committees like Eclipse Adoptium or JakartaEE and actively contributing to even more projects. Hendrik has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Stoyan",
"lastName":"Panayotov",
"gitHubAccount": "https://github.com/stoqnkpL",
"photo": "stoyan.jpg",
"bio": "Stoyan Panayotov is a software architect at Limechain with several years of experience in the blockchain industry consulting various clients - decentralized networks, protocols and dApps. Stoyan has been a core committer to the consensus node of Hedera for several years."
},
{
"firstName": "Alex",
"lastName":"Popowycz",
"gitHubAccount": "https://github.com/popowycz",
"photo": "alex.jpg",
"bio": "Alex is the Chief Information Officer of Hedera and is charged with the technical operations of the network in close partnership with Hedera's member operators, ensuring the sustainable evolution of capabilities to the network while maintaining its core attributes of speed, fairness, and security. Prior to joining Hedera, Alex was the CIO at Health First, a multi-hospital integrated health system. He previously held several technology leadership positions at Fidelity Investments, with primary responsibility for cybersecurity and standards."
},
{
"firstName": "Georgi",
"lastName":"Lazarov",
"gitHubAccount": "https://github.com/georgi-l95",
"photo": "georgi.jpeg",
"bio": "Georgi Lazarov is a technical lead at LimeChain with more than 5 years of experience in the blockchain industry. He is a core committer to the tooling around Hiero, like Local Node and JSON-RPC Relay, as well as a contributor to the Block Node and other core components of the Hiero ecosystem."
},
{
"firstName": "Milan",
"lastName":"Wiercx van Rhijn",
"gitHubAccount": "https://github.com/MilanWR",
"photo": "milan.jpeg",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What Copilot says is wrong and should not be done like that. Pictures are in the public/images/tsc folder

Comment thread nextjs/next.config.ts

const nextConfig: NextConfig = {
/* config options here */
images: {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What is that doing?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

going to work on it when we reach the extent of working on the tsc members ui

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

i also agree with you

Copy link
Copy Markdown
Collaborator

@aceppaluni aceppaluni left a comment

Choose a reason for hiding this comment

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

@danielmarv This is looking good, thank you!!

Note:

  • Review was discussed in discord via DM.

Came to conclusion AI suggestions and additional issues will be addressed in the next PR.

@danielmarv danielmarv merged commit 17f77fa into next-js-migration Mar 11, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants