Skip to content

realization jv-web-practice#662

Open
bigvini wants to merge 10 commits intomate-academy:masterfrom
bigvini:jv-web-practice-solution
Open

realization jv-web-practice#662
bigvini wants to merge 10 commits intomate-academy:masterfrom
bigvini:jv-web-practice-solution

Conversation

@bigvini
Copy link
Copy Markdown

@bigvini bigvini commented Aug 6, 2023

No description provided.

Copy link
Copy Markdown

@okuzan okuzan left a comment

Choose a reason for hiding this comment

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

Please fix the build before requesting another review

car.setModel(model);
car.setManufacturer(manufacturer);
car.setDrivers(new ArrayList<>());
carService.create(car);
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

after post redirect to the page where you can see the changes just made (all cars after adding a car etc)

@bigvini bigvini requested a review from okuzan August 7, 2023 21:10
Copy link
Copy Markdown

@olekskov olekskov left a comment

Choose a reason for hiding this comment

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

Good job

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