Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

Glasgow Class 6 - Christina Mifsud - React - Hotel React #611

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

christina-mifsud
Copy link

No description provided.

@christina-mifsud christina-mifsud changed the title Christina's Branch for Review Glasgow Class 6 - Christina Mifsud - React - Hotel React Jul 3, 2023
Added second state in order to not overwrite the original array when filtering with the search.
// This exercise was a collaborative effort between christina-mifsud and munozirianni1988
@sonarcloud
Copy link

sonarcloud bot commented Jul 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Comment on lines +19 to +26
<td>{booking.id}</td>
<td>{booking.title}</td>
<td>{booking.firstName}</td>
<td>{booking.surname}</td>
<td>{booking.email}</td>
<td>{booking.roomId}</td>
<td>{booking.checkInDate}</td>
<td>{booking.checkOutDate}</td>
Copy link

Choose a reason for hiding this comment

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

consider destructuring the booking object instead of calling it every time

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants