Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajusta validações #128

Closed
wants to merge 28 commits into from
Closed

Conversation

gustavomendess
Copy link
Contributor

Ajusta as validações de registro de conteúdos

gustavomendess and others added 24 commits May 21, 2022 01:39
…de-conteudos

Melhorias registro de conteúdos
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 2, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Member

@edersoares edersoares left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mesmos retornos do PR #126 (review), faz sentido fecharmos o outro não?

@msoares94
Copy link
Contributor

@gustavomendess e @edersoares algum retorno sobre essa melhoria proposta pela comunidade, ela está a bastante tempo aguardando reintegração e é algo muito útil e parametrizável?

O ingresso dessa funcionalidade estimula a contribuição por parte da comunidade, pois sabemos que nossas propostas são aceitas.

@edersoares
Copy link
Member

@msoares94 consegue abrir um novo PR? Desculpe pela demora em fazer o merge mas agora vamos voltar com força em estar mais presente analisando o que comunidade contribui.

@msoares94
Copy link
Contributor

@edersoares essa demanda eu enviei para a comunidade via @gustavomendess, no qual eu realizei o pagamento para ele fazer e enviar o PR direto no repo da comunidade.

@gustavomendess poderia providenciar o que o @edersoares está solicitando.

@edersoares se o @gustavomendess não puder não seria possível seguir com o que tem enviado?

Essa PR é importante e foi feito de forma bem configurável para não impactar instalações já realizadas

@edersoares
Copy link
Member

Entendido @msoares94, acho que fica bom a gente centralizar todos os PRs em apenas 1 e seguirmos a partir dali, pois houve várias mudanças no software enquanto isso. Peço desculpas pela demora, mas este ano foi um pouco agitado por aqui.

@msoares94
Copy link
Contributor

Entendido @edersoares

@gustavomendess consegue atender ao pedido do @edersoares ?

@edersoares
Copy link
Member

Já aceito o PR #129.

@edersoares edersoares closed this Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants