ardalis/Result

errorMessages should be passed as string

Aperta

#124 aperta il 21 mar 2023

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)C# (126 fork)github user discovery
enhancementgood first issuehacktoberfesthelp wanted

Metriche repository

Star
 (1034 stelle)
Metriche merge PR
 (Merge medio 8g 3h) (2 PR mergiate in 30 g)

Descrizione

The error messages get translated into the details property of the problem details object. By the RFC standard this property is a string. Currently a list of errormessages can be passed to the NotFound and Error Method, which then get concatenated. e.g.,: https://github.com/ardalis/Result/blob/1c8a9f53b8995e82af0b1650b5c83ec15060289a/src/Ardalis.Result.AspNetCore/ResultStatusMap.cs#L117-L120 This breaks internationalization, as the string at the start is english and already happens if only a single string is passed. I think the best approach would be to allow only strings and make the developer responsible for concatenation. What do you think? @ardalis @KyleMcMaster

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente dei metodi NotFound e Error in ResultStatusMap.cs, identifica dove vengono concatenati i messaggi di errore e proponi modifiche per accettare solo una singola stringa per conformarsi agli standard RFC e supportare l'internazionalizzazione.
Tech stack
csharp
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75