fly-apps/litefs-js

ensurePrimary doesn't work fine

Open

#6 ouverte le 17 déc. 2023

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)TypeScript (4 forks)github user discovery
help wantedneeds reproduction

Métriques du dépôt

Stars
 (164 stars)
Métriques de merge PR
 (Métriques PR en attente)

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.

Guide contributeur

Direction de recherche
Examinez la fonction `ensurePrimary` dans la bibliothèque litefs js pour comprendre comment elle interagit avec les hooks de navigation dans Remix. Recherchez les effets secondaires asynchrones ou les changements d'état qui pourraient faire recharger le routeur au lieu de naviguer.
Stack technique
typescriptnodejs
Domaine
backendclouddatabase
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
JavaScriptRemix
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60