From 97427f9f6f5502bf2a494ea56f0a6d1fdbc713ec Mon Sep 17 00:00:00 2001 From: Vikrant Date: Sat, 3 Oct 2026 00:14:23 +0530 Subject: [PATCH] Fix CI: register the ReceiptX icon and update the nav test for the new certificate pages The Demand Letter page used the ReceiptX icon, but icons.js only re-exports a curated allowlist from @phosphor-icons/react, and ReceiptX wasn't in it -- failing the nav test that checks every page icon resolves. The other nav test asserting the exact Certificates submenu contents also needed the two new entries added. --- src/ui/icons.js | 2 ++ src/ui/nav/navigation.test.js | 8 ++++++++ 2 files changed, 10 insertions(+) diff --git a/src/ui/icons.js b/src/ui/icons.js index 8270959f3..9f56fe31b 100644 --- a/src/ui/icons.js +++ b/src/ui/icons.js @@ -48,6 +48,7 @@ import { Newspaper, PlusCircle, Receipt, + ReceiptX, Scroll, Shuffle, Signature, @@ -121,6 +122,7 @@ export const ICONS = { Newspaper, PlusCircle, Receipt, + ReceiptX, Scroll, Shuffle, Signature, diff --git a/src/ui/nav/navigation.test.js b/src/ui/nav/navigation.test.js index b6eb5226c..121f95788 100644 --- a/src/ui/nav/navigation.test.js +++ b/src/ui/nav/navigation.test.js @@ -144,6 +144,14 @@ describe("buildNavGroups", () => { label: "Paid / Unpaid Certificate", to: "/certificates/fee-certificate", }), + expect.objectContaining({ + label: "Demand Letter", + to: "/certificates/demand-letter", + }), + expect.objectContaining({ + label: "Fee Structure Certificate", + to: "/certificates/fee-structure-certificate", + }), ], }), ]);