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

Deletar conteúdos por @NahtanN #426

Closed
wants to merge 2 commits into from
Closed

Conversation

filipedeschamps
Copy link
Owner

Isso aqui é o PR de uma branch que o @NahtanN estava trabalhando. Só consegui localizar agora e vou fazer a análise 🤝

Alterações que habilitam a função de excluir postagem, seja ela um comentário ou o post "raiz"

BREAKING CHANGE: Alterações no banco de dados, querys e alguns schemas de validação; além de algumas
mudanças no componente "Content"

feat #348 e feat #349
@vercel
Copy link

vercel bot commented May 31, 2022

@NahtanN is attempting to deploy a commit to the TabNews Team on Vercel.

To accomplish this, @NahtanN needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

Copy link
Owner Author

@filipedeschamps filipedeschamps left a comment

Choose a reason for hiding this comment

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

@NahtanN meus sinceros parabéns pela alteração, você conseguiu ultrapassar a barreira de mexer em todas as camadas do projeto e isso de fato é sensacional!! Não é fácil encarar um projeto assim desconhecido e propor alterações desse tamanho 🤝

Em paralelo, fiz vários comentários ao ponto de talvez ser melhor para o desenvolvimento dessa feature começar aos poucos, por exemplo começar só pela migration e levar isso para produção. E depois incrementalmente as outras features. Acho que esse PR acabou tocando em vários pontos ao mesmo tempo e que afastam a aplicação do design inicial. Fora que revelou outros pontos onde a implementação vai ficar mais complicada do que inicialmente planejado.

models/authorization.js Show resolved Hide resolved
models/content.js Show resolved Hide resolved
models/content.js Show resolved Hide resolved
models/content.js Show resolved Hide resolved
pages/interface/components/Content/index.js Show resolved Hide resolved
pages/interface/components/Content/index.js Show resolved Hide resolved
@filipedeschamps
Copy link
Owner Author

@filipedeschamps filipedeschamps deleted the delete-post-modal branch June 5, 2022 18:14
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.

None yet

2 participants