Skip to content

Catalogue items flagging #1656#1717

Open
joshdimanteto wants to merge 17 commits intodevelopfrom
catalogue-items-flaggging-#1656
Open

Catalogue items flagging #1656#1717
joshdimanteto wants to merge 17 commits intodevelopfrom
catalogue-items-flaggging-#1656

Conversation

@joshdimanteto
Copy link
Contributor

@joshdimanteto joshdimanteto commented Feb 17, 2026

Description

see #1656 , #1658

Testing instructions

Add a set up instructions describing how the reviewer should test the code

  • Review code
  • Check Actions build
  • Review changes to test coverage
  • {more steps here}

Agile board tracking

closes #1656, closes #1658

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

❌ Patch coverage is 94.44444% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 92.96%. Comparing base (686a86b) to head (5dfee95).

Files with missing lines Patch % Lines
src/App.tsx 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1717      +/-   ##
===========================================
- Coverage    92.98%   92.96%   -0.02%     
===========================================
  Files           13       13              
  Lines          627      640      +13     
  Branches       195      199       +4     
===========================================
+ Hits           583      595      +12     
- Misses          44       45       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@joshdimanteto joshdimanteto force-pushed the catalogue-items-flaggging-#1656 branch 2 times, most recently from 9996181 to 6620090 Compare February 17, 2026 15:00
@joshdimanteto joshdimanteto force-pushed the catalogue-items-flaggging-#1656 branch from 6620090 to 692de57 Compare February 17, 2026 15:02
@joshdimanteto joshdimanteto force-pushed the catalogue-catagories-flagging-#1659 branch from 6e666d3 to bca3c9f Compare March 2, 2026 18:08
@joshdimanteto joshdimanteto force-pushed the catalogue-items-flaggging-#1656 branch from d52fda7 to 0ab1735 Compare March 4, 2026 17:02
Base automatically changed from catalogue-catagories-flagging-#1659 to develop March 9, 2026 09:35
@joshdimanteto joshdimanteto force-pushed the catalogue-items-flaggging-#1656 branch from 7185e01 to bd50c90 Compare March 9, 2026 13:11
@joshdimanteto joshdimanteto marked this pull request as ready for review March 9, 2026 13:24
Copy link
Contributor

@joelvdavies joelvdavies left a comment

Choose a reason for hiding this comment

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

Just a couple minor comments/suggestions. Worked well for me and looked good. There is a scrollbar on the items table, due to the new header, but I suspect that will be fixed in the next PR.

@joshdimanteto joshdimanteto force-pushed the catalogue-items-flaggging-#1656 branch from 575dc36 to 766d5ea Compare March 10, 2026 09:17
@joshdimanteto joshdimanteto force-pushed the catalogue-items-flaggging-#1656 branch from 9de3411 to 89d04e5 Compare March 10, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants