fix: project environments order

[Tymek]

About the changes

Environments in /api/admin/projects/$id are not sorted. Having it in correct order from backend will make new Single Project Overview - Features list table easier to implement.

Important files

src/lib/db/project-store.ts

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Updated
unleash-docs	⬜️ Ignored (Inspect)		May 23, 2022 at 8:49AM (UTC)

[sighphyre]

I don't necessarily have a problem with this change but a join and a sort is a bit of a computational cost on the db side. Should be a cold path so this doesn't seem like a problem but I think it's probably worth unpacking a little bit what the pain is on the front end side. I'm happy to approve but I think a third set of eyes would be sensible.

As an aside, I don't think the test that was added here actually ran on the build, I see 868 total on the build output but 869 total on my machine. Pretty sure that's nothing to do with this PR but it's definitely worth understanding

[sighphyre]

Thanks, looks good

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟢	Statements	91.34%	5285/5786
🟡	Branches	79.16%	813/1027
🟢	Functions	86.16%	1226/1423
🟢	Lines	91.27%	5164/5658

Show files with reduced coverage 🔻

St.❔	File	Statements	Branches	Functions	Lines
🟡	... / openapi-service.ts	68.75% (-12.5% 🔻)	12.5% (-50% 🔻)	62.5% (-12.5% 🔻)	68.75% (-12.5% 🔻)

Test suite run success

870 tests passing in 124 suites.

Report generated by 🧪jest coverage report action from 3ace8b0