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

Encja ORDER - Poprawa struktury plików w projekcie #7

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@przemyslawkrawczynski
Copy link
Collaborator

przemyslawkrawczynski commented Feb 2, 2019

No description provided.

@przemyslawkrawczynski przemyslawkrawczynski changed the title Encja Produkt - Poprawa struktury plików w projekcie Encja ORDER - Poprawa struktury plików w projekcie Feb 2, 2019

@pplutap pplutap requested a review from haker-msl Feb 2, 2019

@@ -0,0 +1,4 @@
package com.kodilla.ecommercee.controller;

public class UserController {

This comment has been minimized.

@haker-msl

haker-msl Feb 3, 2019

Collaborator

Ten PR z UserController jeszcze nie został zmerge'owany a już mamy ten kontroller przeniesiony do innego pakietu.
https://github.com/pplutap/JDP-1901-01/pull/4/files
Trzeba to jakoś ogarnąć. Dogadajcie się ze sobą :)


@Override
List<Order> findAll();
@Override

This comment has been minimized.

@haker-msl

haker-msl Feb 3, 2019

Collaborator

pierdoła ale może przydałaby się pusta linia pomiędzy końcem deklaracji jednej metody a początkiem następnej ?

@@ -0,0 +1,4 @@
package com.kodilla.ecommercee.exception;

public class OrderNotFoundException extends Exception {

This comment has been minimized.

@haker-msl

haker-msl Feb 3, 2019

Collaborator

na razie nie widzę miejsca gdzie używasz tego wyjątku... to jest "na przyszłość" ?

@przemyslawkrawczynski przemyslawkrawczynski deleted the Encja-Produkt-PLus-Struktura-Projektu branch Feb 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment