ardalis/Result

errorMessages should be passed as string

Open

#124 aberto em 21 de mar. de 2023

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)C# (126 forks)github user discovery
enhancementgood first issuehacktoberfesthelp wanted

Métricas do repositório

Stars
 (1.034 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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

Guia do colaborador

Direção de pesquisa
Investigue a implementação atual dos métodos NotFound e Error em ResultStatusMap.cs, identifique onde as mensagens de erro são concatenadas e proponha alterações para aceitar apenas uma única string para cumprir os padrões RFC e suportar internacionalização.
Pilha de tecnologia
csharp
Domain
backendapi
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75