Skip to content

Feature 7/registro de clientes#11

Open
tomivillarreal wants to merge 3 commits intoProgramacion-Avanzada-UTN-FRVM:mainfrom
tomivillarreal:feature-7/registro-de-clientes
Open

Feature 7/registro de clientes#11
tomivillarreal wants to merge 3 commits intoProgramacion-Avanzada-UTN-FRVM:mainfrom
tomivillarreal:feature-7/registro-de-clientes

Conversation

@tomivillarreal
Copy link

.

tomivillarreal and others added 2 commits October 22, 2024 11:24
Add Cliente Repository
Add Cliente Service
Add Cliente Controller (Incomplete)
Copy link
Member

@punteroo punteroo left a comment

Choose a reason for hiding this comment

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

.


import com.example.project.management.Model.Cliente;

public interface ClienteRepository extends JpaRepository<Cliente, Integer> {
Copy link
Member

Choose a reason for hiding this comment

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

Eliminar espacios extra.

import com.example.project.management.Repository.ClienteRepository;

@Service
public class ClienteService {
Copy link
Member

Choose a reason for hiding this comment

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

No esta implementado la interfaz de IClienteService.

Copy link
Member

@punteroo punteroo left a comment

Choose a reason for hiding this comment

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

Solucionado, se puede implementar a produccion.

Copy link
Member

@punteroo punteroo left a comment

Choose a reason for hiding this comment

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

Última solicitud antes de mergear, vamos a utilizar MySQL/MariaDB como motor de base de datos por una cuestión de compatibilidad con la cátedra de programación avanzada.

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