search

srivastavaritik / Extraspace-Cloud

A cloud where users can make folders, upload all types of files and links, making it easier to keep everything organised and secure with security of Google's Firebase.
https://extraspace-cloud.netlify.app
MIT License
27 stars 48 forks source link

Readme Updation, Race Conditions, and Security Risks Handled #165

Closed rks-031 closed 10 months ago

rks-031 commented 1 year ago

Issue name: Readme Updation | Race Conditions | Security Risks (for Old Browsers) Issue No: #164

Changes implemented:

  1. The README.md file is updated by adding an additonal section illustrating the steps to follow if the user encounters Cannot find module 'react' error.

readme

  1. In useEffect Callback functions, aync functions have been put inside to prevent race condtions for ContributorsModal.js and useFolder.js file.

corr-2

corr-4

  1. rel="noreferrer" property have been added after target = "_blank" in anchors tags to prevent Security Risks.

corr-5

vercel[bot] commented 1 year ago

Someone is attempting to deploy a commit to a Personal Account owned by @srivastavaritik on Vercel.

@srivastavaritik first needs to authorize it.

netlify[bot] commented 1 year ago

Deploy Preview for extraspace-cloud failed.

Name Link
Latest commit b1a2937db15bfdb4456b4b352af6f62e2c2f929a
Latest deploy log https://app.netlify.com/sites/extraspace-cloud/deploys/64916b0531823000083f8cdd
Dharmesh177 commented 1 year ago

Deployment preview failed kindly check

rks-031 commented 1 year ago

Deployment preview failed kindly check

@Dharmesh177 There are no conflicts with my main branch to your base (master) branch, so there will be no issues in merging my PR. Also, you need to authorize my commit deployement . Kindly look into it.

rks-031 commented 1 year ago

Deployment preview failed kindly check

@Dharmesh177 There are no conflicts with my main branch to your base (master) branch, so there will be no issues in merging my PR. Also, you need to authorize my commit deployement . Kindly look into it.

@Dharmesh177 Kindly look into it.