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

Fix/projects #169

Merged
merged 9 commits into from
May 11, 2024
Merged

Fix/projects #169

merged 9 commits into from
May 11, 2024

Conversation

sgarnica1
Copy link
Contributor

@sgarnica1 sgarnica1 commented May 9, 2024

Fix Proyectos

[Feature/Task ID]: Descripción del cambio

Descripción

Ordenar proyectos por status, evitar que los proyectos se puedan crear para clientes archivados y reemplazar obtener clientes por rol en todas las llamadas

Cambios realizados

  • src/api/controllers/project.controller.ts
  • src/core/app/services/tests/company.service.test.ts
  • src/core/app/services/tests/home.service.test.ts
  • src/core/app/services/tests/project.service.test.ts
  • src/core/app/services/home.service.ts
  • src/core/app/services/project.service.ts
  • src/core/infra/repositories/employee.repository.ts
  • src/core/infra/repositories/employee.repository.ts
  • src/core/infra/repositories/project.repository.ts

Pruebas

image

@sgarnica1 sgarnica1 added the bug Something isn't working label May 9, 2024
@sgarnica1 sgarnica1 added this to the MVP milestone May 9, 2024
@sgarnica1 sgarnica1 self-assigned this May 9, 2024
@ArmandoRosasB ArmandoRosasB self-requested a review May 9, 2024 21:04
ArmandoRosasB
ArmandoRosasB previously approved these changes May 9, 2024
@dembA7 dembA7 requested a review from printchard May 11, 2024 21:37
@printchard printchard merged commit cb0937b into develop May 11, 2024
2 checks passed
@printchard printchard deleted the fix/projects branch May 13, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants