Skip to content

first commit#645

Open
IhorPetrytskyi333 wants to merge 4 commits into
mate-academy:masterfrom
IhorPetrytskyi333:hw-web-practice
Open

first commit#645
IhorPetrytskyi333 wants to merge 4 commits into
mate-academy:masterfrom
IhorPetrytskyi333:hw-web-practice

Conversation

@IhorPetrytskyi333
Copy link
Copy Markdown

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.

Before you create PR with your homework, you need to go through the checklist under the task and correct all the points described there. The mentor will not check the work until the checklist points are corrected

import mate.service.CarService;
import mate.service.DriverService;

@WebServlet(urlPatterns = "/drivers/addDriverToCar/add")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

image

@IhorPetrytskyi333 IhorPetrytskyi333 requested a review from okuzan July 9, 2023 17:57
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.

uncommit .ds_store files, separate controllers into packages

Long driverId = Long.valueOf(req.getParameter("driver_id"));
Car car = carService.get(carId);
Driver driver = driverService.get(driverId);
carService.addDriverToCar(driver,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.

let's redirect here to a page where we can see changes applied

@IhorPetrytskyi333 IhorPetrytskyi333 requested a review from okuzan July 19, 2023 00:44
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 remove these files. Ask for help in the chat if you struggle with this
image

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.

2 participants