Skip to content

docs: sort case studies table by lockfile type and project name#577

Open
kira88-code wants to merge 1 commit into
OWASP:mainfrom
kira88-code:sort-case-studies
Open

docs: sort case studies table by lockfile type and project name#577
kira88-code wants to merge 1 commit into
OWASP:mainfrom
kira88-code:sort-case-studies

Conversation

@kira88-code

Copy link
Copy Markdown

Summary of Changes

Resolved the case studies navigation issue by reordering and grouping the index table rows in website/docs/case-studies/index.md based on the specified lockfile criteria.

  • Lockfile Grouping: Sorted rows strictly by lockfile manager type in the requested sequence: npmpnpmYarn.
  • Alphabetization: Fully alphabetized all project entries within their respective lockfile categories (A-Z).
  • Data Integrity: Left all key findings, markdown links, and descriptions completely unchanged.

Local Verification

  • Successfully ran a local development preview server via npm run start inside the website/ directory to visually verify flawless UI rendering.
  • Successfully verified production builds by running npm run build within the website/ folder, ensuring the Docusaurus configuration compiles cleanly with zero syntax or markdown errors.

Closes #544

@kira88-code

Copy link
Copy Markdown
Author

Hi @sonukapo0r,

I've updated the case studies index table according to the acceptance criteria. The entries are now strictly grouped by lockfile type (npm, pnpm, then yarn) and alphabetized within each group.

I also verified the changes locally using both the development server and a clean production build (npm run build) inside the website directory to ensure Docusaurus compiles perfectly with no markdown errors.

Let me know if this looks good to go!

@sonukapoor

Copy link
Copy Markdown
Collaborator

Looks good - clean sort with no content changes. One small fix needed before merge: the file is missing a newline at the end (shown as \ No newline at end of file in the diff). Could you add a trailing newline to website/docs/case-studies/index.md?

@kira88-code

Copy link
Copy Markdown
Author

Hi @sonukapoor,

Quick update on this PR:

  1. Resolved the latest merge conflict and integrated the new 'CamoFox Browser' case study.
  2. Formatted the table strictly according to the repository's tier-sorting criteria (Grouping by npm -> pnpm -> Yarn, and fully alphabetizing entries within each section).
  3. Ensured the trailing newline fix is completely intact to clear the linter check.

All CI builds, security scans, and tests are passing cleanly now. Ready for final review!

@sonukapoor

Copy link
Copy Markdown
Collaborator

Just a reminder - still needs two things before merge: a trailing newline at the end of website/docs/case-studies/index.md, and a rebase against main. Both are quick fixes!

@sonukapoor

Copy link
Copy Markdown
Collaborator

This branch is behind main — could you rebase against main and force-push? Thanks!

@kira88-code

Copy link
Copy Markdown
Author

Hi @sonukapoor,

I’ve pulled the latest upstream changes, rebased the branch, and integrated the new entry into the sorted table.

The table is now fully grouped, alphabetized, and includes the trailing newline. Ready for your review!

@sonukapoor sonukapoor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sorting looks correct — npm first, then pnpm, then Yarn Classic, alphabetical within each group. Just needs a rebase against main before we can merge.

@sonukapoor

Copy link
Copy Markdown
Collaborator

Approved — please rebase against main and force-push, then we can merge.

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.

ux: sort case studies index table by lockfile type

2 participants