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

Full-Stack Capstone - Book an appointment application. #14

Merged
merged 76 commits into from
Dec 14, 2023
Merged

Conversation

tan12082001
Copy link
Owner

@tan12082001 tan12082001 commented Dec 12, 2023

Final group capstone - Book an Appointment

Team ( 4 memebers ):

Project Information:

NOTE: Working on two separate Repositories.

Vehicle-Booking-App-Backend

Vehicle-Booking-App-Frontend-Review-Pull-Request

Instructions for Running the local version:

- npm install
- npm start

note: run `rails s` for API  endpoints.

Implemented Features:

  • Created user authentication - user login and signup.
  • Created a Navigation panel that consists of links to respective pages.
  • Created a Dashboard that lists all the cars present in the database (Did not add any sample data).
  • Created Item Details page which can be viewed by clicking on the specific item from the dashboard.
  • Created a Reserve form that would create reservations for the item (have to click on the Reserve button inside the item details page.
  • Created Add new car page (to add new cars to the endpoint).
  • Created Delete Item list - renders names of all the cars present (not deleted items) to be soft deleted(can be accessible from the database).
  • Created API Documentation.
  • Applied basic styles for both mobile and desktop versions.

Kanban progress views:

Kanban board initial view:

image

Kanban board final view:

image

tan12082001 and others added 30 commits December 1, 2023 19:35
create empty files for folder structure
(not tested as api endpoint is not accessable)
 feature and update store with reducer.
create a form component for reservat car
(works - able to login to an existing user, able to create a new user)
Copy link

@Meltrust Meltrust left a comment

Choose a reason for hiding this comment

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

Hi,

Good job so far!
There are some issues that you still need to work on to prepare your project for the final evaluation, but you are almost there!

To highlight:

  • Tests are passing✔️

You are really close to finishing the Microverse program!! Keep it up! 👍👍👍

You can do it

After implementing the requested changes, please submit another review request. ♻️

Check the comments under the review.

Cheers and Happy coding!👏👏👏

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the previous reviews unless it is requested otherwise.


src/components/DeleteCars/DeleteList.js Outdated Show resolved Hide resolved
src/layout/UsersDashboard/sideNav/navConfig.js Outdated Show resolved Hide resolved
src/layout/UsersDashboard/UsersDashboardLayout.js Outdated Show resolved Hide resolved
src/models/car.model.js Outdated Show resolved Hide resolved
src/pages/LandingPage/Home/Home.js Outdated Show resolved Hide resolved
src/components/DeleteCars/DeleteList.js Show resolved Hide resolved
@@ -0,0 +1,144 @@
* {

Choose a reason for hiding this comment

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

  • The mobile view has not been implemented yet. I understand this is a work in progress or a future update.

image

tip: If you feel you've completed 90% of the features, kindly say in your PR description what features will be left out to give a heads-up to the final evaluator and reviewers.

tip 2: If you declare in your description that the mobile view will be left out, I will still approve the project, however, I don't recommend leaving it undone for final evaluation because this is a huge feature. The final evaluator will make the final decision.

tip 3: If you decide to implement it, you could do so in a simple, but functional way in order to improve your chances (MVP).

tip 4: When dealing with mobile views, the flex column is our best, simple, fast friend. Just make sure the content fits the screen and the navbar does not get in the way

src/App.js Outdated
import Router from './routes/routes';
import { fetchCarReservations, fetchCars } from './redux/thunk';

function App() {

Choose a reason for hiding this comment

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

  • The images of the cars come up as null:

image

image

Kindly, make it so the images of the cars show up. 👍

src/pages/Auth/SignIn/SignIn.js Show resolved Hide resolved
@tan12082001 tan12082001 mentioned this pull request Dec 13, 2023
Copy link

@Meltrust Meltrust left a comment

Choose a reason for hiding this comment

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

Hi,

Good job so far!
There are some issues that you still need to work on to prepare your project for the final evaluation, but you are almost there!

You are really close to finishing the Microverse program!! Keep it up! 👍👍👍

You can do it

After implementing the requested changes, please submit another review request. ♻️

Check the comments under the review.

Cheers and Happy coding!👏👏👏

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the previous reviews unless it is requested otherwise.


@@ -0,0 +1,308 @@
<div align="center">

Choose a reason for hiding this comment

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

  • One of the frontend tests is no longer working, kindly fix it so it passes. 👍

image

src/App.js Show resolved Hide resolved
@@ -0,0 +1,266 @@
* {

Choose a reason for hiding this comment

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

  • OPTIONAL: The mobile view home page is very questionable:

image

Also, the maximum width seems to be 375px which is tiny for almost every phone. I suggest you raise that number so more phones can enjoy your work.

this is optional because in the last review you said you've completed 90% of the features.

import Router from './routes/routes';

const App = () => (
<>

Choose a reason for hiding this comment

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

  • OPTIONAL: If I reload the page, all the information disappears from all sections.

The only way to display the information again is to logout and login again. I suggest you look into this issue.

@tan12082001 tan12082001 mentioned this pull request Dec 14, 2023
Copy link

@Meltrust Meltrust left a comment

Choose a reason for hiding this comment

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

Wow, you did it 🎉

Brilliant

Thank you for the changes implemented 💪 🥇 ㊗️

Unless you want to add more features, go ahead to your final presentation ⏩ ⏩ ⏩

You are about to finish the Microverse program. You have come a long way!!!

Good luck in the software industry!! I'll see you there. ✨

Congratulations!!!!!! 🎉

applause

To highlight

  • Superb job ✔️

Cheers and Happy coding!👏👏👏


@tan12082001 tan12082001 merged commit 228f017 into main Dec 14, 2023
3 checks passed
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.

4 participants