Sign-up errors can show participants the server's English text
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- backend, frontend, internationalization
Direzione di ricerca
Read src/app/s/[slug]/participant-error.ts, src/services/commitments.ts, src/lib/errors.ts and src/lib/api-response.ts; first trace how error codes and details reach the signup dialog and edit form. Then map server errors to reasons and catalog messages while preserving the API's English message and suggestion. Done means every participant error has English and Swedish catalog text, with the listed checks and database tests passing.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Needs your OK: this makes every error a participant can see come from the catalog, picked by error code, instead of from the server's English. Swap pipeline:proposed for pipeline:ready to have it built.
What happens
src/app/s/[slug]/participant-error.ts maps most error codes to its own copy, but some server text still reaches the screen word for word:
- Missing email. "Email is required for this signup." / "Add your email and try again." (
src/services/commitments.ts). - Zod's defaults. "Invalid email" and "Number must be less than or equal to 999", with a suggestion built in
src/lib/errors.ts(`Check the value of "${path}".`). - The edit page shows
rate_limited"too many requests",forbidden"invalid edit token" andconflict"commitment is not active". - Fallbacks typed in the client: 'save failed', 'cancel failed', 'something went wrong' and 'network error'.
Fix, in outline
- Add a
details.reasonwhere a code is too coarse:email_required,email_invalid,quantity_too_large, and whatever the edit page needs. - Map Zod issues by field and issue code to reasons on the server, rather than passing
issue.messagethrough. - Map code plus reason to catalog messages in the dialog and the edit form, and make the client fallbacks messages too.
- Keep the API's
messageandsuggestionin English for API callers.
Watch for
- The error shape is shared.
src/lib/api-response.tsalso serves the MCP tools, so addingdetails.reasonmustn't change existing fields. - Tests that pin today's English messages will need updating where the participant now sees catalog text.
Order
After #629, because the participant catalog and the translator-taking helpers come from it.
Done when
- Every error path in the sign-up dialog and the edit form renders catalog text, in English and Swedish (unit tests per code and reason).
- No server
messagereaches the participant's screen. pnpm lint && pnpm typecheck && pnpm knip && pnpm testandpnpm test:dbpass.
Planned with the owner in https://claude.ai/code/session_01EFKmj716ZY3b3SwR8TV8Y2 and checked against the code on main at 35288fa. The full plan is private to the owner's account: https://claude.ai/artifact/VgQH6U2s2EVx3LX991fJi8 (a Claude session on that account can open it with the Artifact tool's read action). This is step 1.3 there.
- Lingua principale
- TypeScript
- Stelle
- 19
- Fork
- 4
- Merge medio
- 21h 3m
- PR unite (30g)
- 176
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di richshaw/OpenSignup
-
enhancement pipeline:proposed pipeline:ux
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
richshaw/OpenSignup#654 ·
I maintainer di solito rispondono entro 1 giorno
-
The help pages never say "pause", so organizers looking for it miss Close or reopen a signupForse già presa @richshaw l’ha presa oggi. Apertaenhancement pipeline:building pipeline:ux
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
richshaw/OpenSignup#653 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement pipeline:proposed pipeline:ux
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
richshaw/OpenSignup#652 ·
I maintainer di solito rispondono entro 1 giorno
-
SMTP timeout notes in smtp.ts and the changelog describe nodemailer 10.0, not 10.1Forse già presa @agammann l’ha presa 2 giorni fa. Apertaenhancement good first issue pipeline:ready
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
richshaw/OpenSignup#645 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement pipeline:proposed pipeline:ux
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
richshaw/OpenSignup#589 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di richshaw/OpenSignup
Issue simili
-
Plugin: sheet music can't be scrolled by hand before playbackForse già presa @snakajima l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
receptron/jinglescript#51 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 9 giorni
-
clawsweeper:needs-maintainer-review clawsweeper:needs-product-decision clawsweeper:no-new-fix-pr clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
openclaw/openclaw#186424 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
chore(worker): five module header comments describe the code from before later issues changed itApertaarea:dev U2-later
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
bentleypark/aiwatch#1824 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
Pal-AI-Lab/Coopanion#469 ·
I maintainer di solito rispondono entro 1 giorno