fly-apps / fly-apps/litefs-js

ensurePrimary doesn't work fine

Open
#6 3 comments 0 reactions 0 assignees View on GitHub
help wanted needs reproduction
Dominant language
TypeScript
Stars
164
Forks
4
PR merge metrics
No merged PRs in 30d

Description

I had a route which I put await `ensurePrimary()` on the loader because I was doing a update() request with prisma, and including `await ensurePrimary()` on the loader, was making all the links to that loader (both with useNavigate and remix Link) to reload the current page instead of go to that new page.

So I ended removing `ensurePrimary()`, and the links worked again.

Contributor guide

Open the contributing guide

Research direction

Start at the ensurePrimary() utility and reproduce the reported behavior in a route loader that awaits it before a Prisma update. Compare navigation through useNavigate and Remix Link with and without ensurePrimary(); done means both links reach their intended new pages while the loader still awaits the primary check.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
distributed-systems
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.