Skip to content

Comments

Eliminação da entidade Perfil de legacy e correção das chamadas da mesma#61

Open
DouglasRbto wants to merge 23 commits intomainfrom
dr-hotfix-eliminaPerfilDeLegacyModels
Open

Eliminação da entidade Perfil de legacy e correção das chamadas da mesma#61
DouglasRbto wants to merge 23 commits intomainfrom
dr-hotfix-eliminaPerfilDeLegacyModels

Conversation

@DouglasRbto
Copy link
Contributor

Descrição

Foi eliminada a classe Perfil.cs da pasta legacy e refatorado o código em todos os locais que a utilizavam para que use a nova que se encontra em Domain.Entities.

Tive que alterar a classe para que tenha campos opcionais para que consiga instanciar em CustomPageModel, mas não sei se foi a melhor solução, então favor me corrigir para que eu possa refazer, caso necessário.

Checklist antes de compartilhar o PR no grupo

  • Escrevi testes que garantem que minha alteração funciona como esperado
  • Executei o comando dotnet csharpier . na raiz do projeto
  • Garanti que meu código não gera novos warnings ou alertas do Sonar Cloud
  • Revisei extensivamente a versão final do meu código analisando as alterações que estão entrando e saindo

@DouglasRbto DouglasRbto self-assigned this Aug 11, 2024
@DouglasRbto DouglasRbto added the .NET Pull requests that update .net code label Aug 11, 2024
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 7, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
10.4% Duplication on New Code (required ≤ 3%)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

.NET Pull requests that update .net code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants