Skip to content

Feature/folder owner#22

Merged
jhonatapers merged 11 commits intodevelopfrom
feature/folder-owner
Mar 19, 2025
Merged

Feature/folder owner#22
jhonatapers merged 11 commits intodevelopfrom
feature/folder-owner

Conversation

@MatheusmPigatto
Copy link
Copy Markdown
Contributor

@MatheusmPigatto MatheusmPigatto commented Mar 10, 2025 •
edited
Loading

Descrição

Tipo de mudança

  • Bug fix (correção de um problema)
  • Nova funcionalidade (adição de uma nova feature)
  • Refatoração (melhoria de código sem alterar funcionalidades)
  • Documentação (adição ou atualização de documentação)
  • Outro (especifique):

Como foi testado?

  • Testes unitários
  • Testes de integração
  • Testes manuais
  • Nenhum teste realizado

Checklist

  • Meu código segue as diretrizes do projeto
  • Eu realizei uma auto-revisão do meu código
  • Eu fiz as mudanças correspondentes na documentação
  • Todos os testes existentes e novos passaram

Informações adicionais

@MatheusmPigatto
Copy link
Copy Markdown
Contributor Author

MatheusmPigatto commented Mar 10, 2025 •
edited
Loading

Precisamos adicionar a nova coluna no db

ALTER TABLE folders ADD COLUMN owner_id VARCHAR(255);

jhonatapers
jhonatapers reviewed Mar 11, 2025
View reviewed changes
Comment thread domain/src/main/java/com/callv2/drive/domain/folder/Folder.java Outdated
jhonatapers
jhonatapers reviewed Mar 11, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@jhonatapers jhonatapers left a comment

Choose a reason for hiding this comment

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

Alterar caso de uso do GetRootFolderUseCase para contemplar a lógica de owner também.

Adicionar ao MemberGateway função de buscar existência, algo como "existsById"

jhonatapers

This comment was marked as duplicate.

Sign in to view

@jhonatapers jhonatapers marked this pull request as draft March 11, 2025 19:54
@jhonatapers jhonatapers linked an issue Mar 11, 2025 that may be closed by this pull request
Owner de pastas #18
Closed
@jhonatapers jhonatapers self-assigned this Mar 11, 2025
@MatheusmPigatto MatheusmPigatto marked this pull request as ready for review March 19, 2025 00:27
@jhonatapers jhonatapers merged commit 500fa4b into develop Mar 19, 2025
@jhonatapers jhonatapers deleted the feature/folder-owner branch March 23, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jhonatapers jhonatapers jhonatapers approved these changes

Assignees

@jhonatapers jhonatapers

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Owner de pastas

2 participants

@MatheusmPigatto @jhonatapers