search

Team-Roots / voluntree-meteor-react

MIT License
0 stars 0 forks source link

Review: RegistartionOrganization.jsx and ViewOrganization.jsx #143

Open kristineorpilla03 opened 6 months ago

kristineorpilla03 commented 6 months ago

Overview

The focus for this code review will be centered around the files in this project. Please pay attention too:

Review Branch

review-7

Files to review

Checklists

Due date

03/11

For more information

The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

kristineorpilla03 commented 6 months ago

RegistartionOrganization.jsx

L28: The age range is hardcoded to be 18-99; I think this should be something organizations are able to change this when registering

ViewOrganization.jsx

Consistency with the format, where if netting, choosing between having it all on one line or on multiple lines.

colas002 commented 6 months ago

RegistartionOrganization.jsx

No error found

ViewOrganization.jsx

L13: comment could start at the new line

UHMyafeiwang commented 6 months ago

RegistartionOrganization.jsx

No error found

ViewOrganization.jsx

JS-07 : No console logs present

tliam1 commented 6 months ago

RegistartionOrganization.jsx

josiahkila commented 6 months ago

RegistartionOrganization.jsx

No error found

ViewOrganization.jsx

formatting issues

juvyannl commented 5 months ago

RegistrationOrganization.jsx

No error found

ViewOrganization.jsx

L13: new line needed

destinyshishido commented 5 months ago

RegistrationOrganization.jsx

No errors

ViewOrganization.jsx

L13 add newline