sveltekit error handling

[eyeseast]

#502

This wraps API responses in a new type and abstracts handling responses from the API. It uses typescript generics to pass through the types we need, for both data and errors.

[github-actions]

Coverage Report

Status	Category	Percentage	Covered / Total
🔵	Lines	23.19%	6305 / 27183
🔵	Statements	23.19%	6305 / 27183
🔵	Functions	40.18%	133 / 331
🔵	Branches	57.58%	334 / 580

File Coverage

File	Stmts	% Branch	% Funcs	% Lines	Uncovered Lines
Changed Files
src/lib/api/addons.ts	66.93%	94.73%	54.54%	66.93%	71-80, 85-102, 107-119, 126-145, 150-169
src/lib/api/collaborators.ts	100%	100%	100%	100%
src/lib/api/documents.ts	97.88%	88.67%	100%	97.88%	130-131, 148-154, 158-159, 458-460
src/lib/api/flatpages.ts	0%	0%	0%	0%	1-15
src/lib/api/notes.ts	85.78%	78.57%	83.33%	85.78%	25-37, 43-58
src/lib/api/projects.ts	90.84%	100%	92.3%	90.84%	253-278
src/lib/api/sections.ts	70.66%	100%	60%	70.66%	17-36, 44-67
src/lib/api/types.d.ts	0%	0%	0%	0%	1-293
src/lib/components/forms/DocumentUpload.svelte	100%	25%	0%	100%
src/lib/components/forms/EditNote.svelte	0%	0%	0%	0%	1-107
src/lib/components/forms/EditSectionRow.svelte	0%	0%	0%	0%	1-175
src/lib/components/forms/Reprocess.svelte	0%	0%	0%	0%	1-178
src/lib/components/layouts/AddOnBrowser.svelte	0%	0%	0%	0%	1-174
src/lib/components/layouts/DocumentBrowser.svelte	0%	0%	0%	0%	1-2327
src/lib/components/projects/ProjectPin.svelte	0%	0%	0%	0%	1-55
src/lib/utils/api.ts	95.62%	85.18%	100%	95.62%	93-96, 117-118
src/routes/(app)/add-ons/+page.svelte	0%	0%	0%	0%	1-26
src/routes/(app)/add-ons/[owner]/[repo]/+page.server.ts	0%	0%	0%	0%	1-35
src/routes/(app)/add-ons/[owner]/[repo]/+page.ts	0%	0%	0%	0%	1
src/routes/(app)/add-ons/[owner]/[repo]/[event]/+page.server.ts	0%	0%	0%	0%	1-40
src/routes/(app)/add-ons/[owner]/[repo]/[event]/+page.ts	0%	0%	0%	0%	1
src/routes/(app)/documents/+layout.ts	0%	0%	0%	0%	1
src/routes/(app)/documents/+page.server.ts	0%	0%	0%	0%	1
src/routes/(app)/documents/+page.svelte	0%	0%	0%	0%	1-68
src/routes/(app)/documents/+page.ts	0%	0%	0%	0%	1
src/routes/(app)/documents/[id]-[slug]/+layout.ts	0%	0%	0%	0%	1
src/routes/(app)/documents/[id]-[slug]/+page.server.ts	0%	0%	0%	0%	1
src/routes/(app)/documents/projects/+page.ts	0%	0%	0%	0%	1
src/routes/(app)/documents/projects/[id]-[slug]/+page.server.ts	0%	0%	0%	0%	1-104
src/routes/(app)/documents/projects/[id]-[slug]/+page.ts	0%	0%	0%	0%	1
src/routes/(app)/upload/+page.ts	0%	0%	0%	0%	1
src/routes/embed/documents/[id]-[slug]/+layout.ts	0%	0%	0%	0%	1
src/routes/embed/documents/[id]/annotations/[note_id]/+page.ts	0%	0%	0%	0%	1
src/routes/embed/documents/[id]/pages/[page]/+page.ts	0%	0%	0%	0%	1
src/routes/embed/projects/[project_id]/+page.ts	0%	0%	0%	0%	1

Generated in workflow #474

[netlify]

✅ Deploy Preview for documentcloud-frontend ready!

Name	Link
🔨 Latest commit	f6993cf
🔍 Latest deploy log	https://app.netlify.com/sites/documentcloud-frontend/deploys/66f4287074b58c00082b2388
😎 Deploy Preview	https://deploy-preview-705.muckcloud.com
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[allanlasser]

Some initial guidance on generic types before you go further on retyping API calls. Overall makes a ton of sense.

[allanlasser]

Caught one minor issue. And a question: are +error.svelte pages coming later?