Skip to content

Conversation

@addiebart
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings February 9, 2026 02:10
@addiebart addiebart merged commit d1dc4dc into master Feb 9, 2026
2 checks passed
Copy link

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

Removes the temporary IPC 2026 navigation entry so the IPC page is no longer linked from the site header, while also introducing a backup copy of the IPC page content and adjusting tag rendering in the archive listing.

Changes:

  • Removed the time-gated “IPC 2026” header navigation link.
  • Added an ipc26.astro.bak backup page containing IPC 2026 content.
  • Modified archive tag rendering logic to include a "None" fallback (currently ineffective).

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/pages/ipc26.astro.bak Adds a backup copy of IPC 2026 page content (not routed as .astro, but currently contains invalid markup).
src/navigation.js Removes the temporary IPC 2026 link from the header navigation.
src/components/ArchiveListing.tsx Changes tag rendering expression to add a fallback string (but the fallback won’t actually render as written).

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

Comment on lines +59 to 60
{item.tags?.map((e, idx) => <p key={idx} className="p-1 bg-slate-600 rounded-md">{e}</p>) || "None"}
</div> : <></>}
Copy link

Copilot AI Feb 9, 2026

Choose a reason for hiding this comment

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

item.tags?.map(...) || "None" will never render "None": map always returns an array (truthy), and this branch only runs when item.tags.length > 0. If you want a fallback, render it when tags are missing/empty (e.g., move the fallback to the false branch) rather than using || here.

Suggested change
{item.tags?.map((e, idx) => <p key={idx} className="p-1 bg-slate-600 rounded-md">{e}</p>) || "None"}
</div> : <></>}
{item.tags.map((e, idx) => <p key={idx} className="p-1 bg-slate-600 rounded-md">{e}</p>)}
</div> :
<div className="flex flex-row gap-x-2 flex-wrap">
<p>Tags:</p>
<p className="p-1 bg-slate-600 rounded-md">None</p>
</div>}

Copilot uses AI. Check for mistakes.
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.

1 participant