search

Team-Roots / voluntree-meteor-react

MIT License
0 stars 0 forks source link

Review: ViewOrganizations.jsx and SearchOrganizations.jsx #166

Open kristineorpilla03 opened 5 months ago

kristineorpilla03 commented 5 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-11

Files to review

Checklists

Due date

04/15

For more information

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

choytr commented 5 months ago

ViewOrganization.jsx DE-05: line 64 remove chatgpt comment JS-01: const is named VolunteerOrganizations instead of ViewOrganization

UHMyafeiwang commented 5 months ago

ViewOrganizations.jsx

JS-08 : document complex parts of the component

SearchOrganizations.jsx

No error find

kristineorpilla03 commented 5 months ago

ViewOrganization.jsx

Replace placeholder text with data from collection (mission statement, description, tags) Implement the upcoming events section of the organization

SearchOrganizations.jsx

Implement categories of the organization

colas002 commented 5 months ago

ViewOrganization.jsx

RE-08 : the file is named ViewOrganization, but its component is called VolunteerOrganizations Replace the placeholder text with real data from collection

SearchOrganizations.jsx

no error founded

juvyannl commented 5 months ago

ViewOrganization.jsx

JS-01: Change VolunteerOrganizations to ViewOrganization

SearchOrganization.jsx

No errors found

tliam1 commented 5 months ago

ViewOrganization.jsx

Nothing of importance to add

SearchOrganization.jsx

Nothing of importance to add

destinyshishido commented 5 months ago

ViewOrganization.jsx

Rename VolunteerOrganization to ViewOrganization

SearchOrganization.jsx

No errors?

josiahkila commented 5 months ago

ViewOrganization.jsx DE-05: line 64 remove comment