Display Node ID

[stefanbitcr]

📝 Description

Display Ebill identity data

(Requires Wildcat to expose identity endpoint, latest dev version)

---

Screenshot

---

✅ Checklist

Please ensure the following tasks are completed before requesting a review:

• My code adheres to the style guidelines of this project.
• I have run npm run lint or the equivalent linting command.
• I have added or updated tests (if applicable).
• My changes have been tested thoroughly in different browsers/resolutions (if applicable).
• I have updated the documentation (if applicable).
• I have checked that there are no console errors or warnings.
• I have verified that the application builds without errors.
• I have tested responsiveness for mobile and desktop views (if applicable).

[qodo-code-review]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Error Handling The component lacks proper error handling for the API call. If the identityDetail API fails, the user will see an unhelpful error state. Consider adding error boundaries or try-catch blocks with user-friendly error messages. const { data } = useSuspenseQuery({ queryKey: ["identity-detail"], queryFn: async () => { const response = await identityDetail() return response.data }, staleTime: Infinity, gcTime: Infinity, }) Data Validation The postal_address object is accessed without null checking its properties. While the parent object is checked for null, individual properties like address, city, zip, and country could potentially be undefined, leading to runtime errors. <div className="font-medium">{data.postal_address.address}</div> <div className="text-muted-foreground"> {data.postal_address.city} {data.postal_address.zip && `, ${data.postal_address.zip}`} </div> <div className="text-muted-foreground">{data.postal_address.country}</div>

[Copilot]

Pull Request Overview

This PR replaces the static placeholder in HomePage with a real identityDetail query and adds structured UI cards to display identity information.

• Integrate identityDetail API call via useSuspenseQuery (with infinite cache settings).
• Render separate cards for Information (name, email, DOB), Keys (node ID, Bitcoin key, Nostr key), and Address.
• Remove commented-out placeholders and adjust imports.

Comments suppressed due to low confidence (1)

src/pages/home/HomePage.tsx:15

• Add unit tests for PageBody covering cases like no data (renders 'No identity found') and each optional field (email, date_of_birth, postal_address) to ensure correct rendering.

function PageBody() {

[qodo-code-review]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Use consistent client fallback pattern Use the options.client fallback pattern like other functions in this file. This ensures consistency and allows client override when needed. src/generated/client/sdk.gen.ts [161-170] export const identityDetail = <ThrowOnError extends boolean = false>(options?: Options<IdentityDetailData, ThrowOnError>) => { - return (_heyApiClient).get<IdentityDetailInfo, unknown, ThrowOnError>({ + return (options?.client ?? _heyApiClient).get<IdentityDetailInfo, unknown, ThrowOnError>({ url: '/v1/admin/identity/detail', ...options, headers: { 'Content-Type': 'application/json', ...options?.headers } }); }; [To ensure code accuracy, apply this suggestion manually] Suggestion importance[1-10]: 6 __ Why: This is a good suggestion that improves consistency with other functions in the file, like requestToMint, by allowing the API client to be overridden.	Low
Update

[cloudflare-workers-and-pages]

Deploying wildcat-dashboard with    Cloudflare Pages

Latest commit:	40b462d
Status:	✅  Deploy successful!
Preview URL:	https://4260d316.wildcat-dashboard.pages.dev
Branch Preview URL:	https://stefan-nodeid.wildcat-dashboard.pages.dev

View logs

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 82 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/pages/home/HomePage.tsx	0.00%	72 Missing ⚠️
src/generated/client/sdk.gen.ts	0.00%	10 Missing ⚠️

📢 Thoughts on this report? Let us know!

[zupzup]

LGTM 👍