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 - Booking an Appointment Application. #33

Merged
merged 131 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-Frontend

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

Instructions for Running the local version:

- bundle install
- secret_base_key generation
* rm credentials.yml.enc
* $ EDITOR="mate --wait" bin/rails credentials:edit

- configure database
* run `rails db:create`
* run `rails db:migrate`

- Rspec
* bundle exec rails db:schema:load RAILS_ENV=test
* rspec spec

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

After External review:

image

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✔️
  • Good readme ✔️

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.


- [Ruby and Rails](https://gorails.com/setup/windows/10)
- [Git](https://git-scm.com/book/en/v2/Getting-Started-Installing-Git)

### Setup

Choose a reason for hiding this comment

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

  • Thank you for adding specific instructions about the procedures needed to run setup the app correctly:

image

Kindly, add those instructions to your readme so all your users are able to setup and run your project smoothly. 👍

README.md Outdated
Comment on lines 104 to 105
### Run tests
There are no tests to run.

Choose a reason for hiding this comment

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

  • You have included some functional tests:

image

Kindly, add a "tests" section on your readme with instructions about how to run them correctly. 👍

Open [http://localhost:3000](http://localhost:3000/api/random_greeting) to view it in your browser.


### Run tests

Choose a reason for hiding this comment

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

  • You are using Rspec, but the "test" folder still exists, which is used by another test framework. Kindly consider deleting that folder if you are not using it. 👍

@@ -0,0 +1,31 @@
# See https://docs.docker.com/engine/reference/builder/#dockerignore-file for more about ignoring files.

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.

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.

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,190 @@
<a name="readme-top"></a>

Choose a reason for hiding this comment

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

  • Please check the frontend review for the comments. 👍

Copy link
Owner Author

@tan12082001 tan12082001 Dec 14, 2023

Choose a reason for hiding this comment

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

  • Please check the frontend review for the comments. 👍

Implemented the changes:

  • Added responsive styles.

TestCases:
image

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 bd4af51 into main Dec 14, 2023
2 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.

5 participants