Add manual triage view - Githubissues

Nix-Security-WG / nix-security-tracker

Web service for managing information on vulnerabilities in software distributed through Nixpkgs

26 stars 4 forks source link

Closed alejandrosame closed 4 weeks ago

alejandrosame commented 1 month ago

This view allows users to filter CVEs and packages, link them and create NixpkgsIssue from their selections.

Model changes:

Added SearchVectorField to improve performance of full-text search.
Indices added for performance improvements:
- GinIndex for SearchVectorFields.
- BTreeIndex for fields to be searched for filtering and aggregation.
Part of https://github.com/Nix-Security-WG/nix-security-tracker/issues/5

proofconstruction commented 1 month ago

Some notes:

Can you share a screenshot of the rendered view?
Code-wise this looks OK, but it would be nice to have more comments explaining what things do and why they exist.
Is there any reasonable way to split the big commit into smaller ones?
Is there a reasonable way to roll back these commits, fast-forward the branch to the latest main, and then apply the commits, instead of having that merge commit in the middle?

fricklerhandwerk commented 4 weeks ago

Please rewrite history and I'll merge.

alejandrosame commented 4 weeks ago

Please rewrite history and I'll merge.

Doing it after this last commit. Now there's some basic checkbox state management, but I'm skipping the CSS/HTML change suggestions