Skip to content

Conversation

@Irirwanirira
Copy link
Collaborator

@Irirwanirira Irirwanirira commented Mar 16, 2023

This Pr request consists of:

Link to the Front end Pull Request

  • Fetch API created from the backend side
  • Create React components
  • Use the redux toolkit to fetch data
  • Add functionalities that allow the user to add books
  • Add functionality that allows a user to delete a book
  • Add functionality That allows users to reserve a book

Thank you for Reviewing our project

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 @Irirwanirira ,

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:

  • Nice code organization ✔️
  • App is working well✔️
  • Good readme ✔️
  • Nice frontend implementation✔️

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.


import Input from './reusable/inputFields/input';
import isUserSigned from '../../helpers/auth';

const AddBook = () => {
Copy link

@Meltrust Meltrust Mar 16, 2023

Choose a reason for hiding this comment

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

  • When I want to add a picture to a book that I'm creating, if the image is high resolution it will overflow the frontend containers:

image

This issue is more prominent in the mobile view.

Kindly, add some styling so the image has limits and it does not overflow. 👍

Copy link
Collaborator

Choose a reason for hiding this comment

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

done.

README.md Outdated
Comment on lines 144 to 150
### Run tests

To run tests, run the following command:

```sh
rspec spec/
```
Copy link

@Meltrust Meltrust Mar 16, 2023

Choose a reason for hiding this comment

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

  • This command is meant for the backend project. Since this is the frontend project, kindly remove it. 👍

Copy link
Collaborator

Choose a reason for hiding this comment

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

Omitted accordingly.

Comment on lines 1 to 5
/* .backdrop {
display: flex;
flex-direction: column;
}
*/
Copy link

@Meltrust Meltrust Mar 16, 2023

Choose a reason for hiding this comment

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

  • Kindly, remove this commented code in order to improve the repo's cleanliness. 👍

Copy link
Collaborator

Choose a reason for hiding this comment

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

Removed.

Choose a reason for hiding this comment

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

  • Kindly, remove the commented code present in this file in order to improve the repo's cleanliness. 👍

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 @Irirwanirira,

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:

  • Nice code organization ✔️
  • App is working well✔️
  • Good readme ✔️
  • Beautiful frontend ✔️

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.



<p align="right">(<a href="#readme-top">back to top</a>)</p>

<!-- SUPPORT -->

Choose a reason for hiding this comment

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

  • Since the last review, this pull request has not suffered any changes or commits. Kindly implement the comments featured on the last review. 👍

@netlify
Copy link

netlify bot commented Mar 19, 2023

Deploy Preview for fastidious-biscotti-61dbed ready!

Name Link
🔨 Latest commit a4963b2
🔍 Latest deploy log https://app.netlify.com/sites/fastidious-biscotti-61dbed/deploys/641acde90ef2da0008160aa4
😎 Deploy Preview https://deploy-preview-17--fastidious-biscotti-61dbed.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

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.

6 participants