Content of the controller directory

FAC-11 / AllAboutMe

An app to help young people at Anna Freud centre to easily transition to adult care

https://allaboutme-af.herokuapp.com/

10 stars 6 forks source link

Content of the controller directory #177

Closed sohilpandya closed 6 years ago

sohilpandya commented 6 years ago

The controller directory should contain information related to the controllers that are being handled by endpoints, in our case I've found a few files that don't deal with that scenario.

passwordModule
helper
validate
check_email

Where do you think these files belong?

polyccon commented 6 years ago

@sohilpandya Thanks for raising an issue we were already planning some restructuring of the "controllers" directory and move the above files into a "helpers" one.

What about the requireLogin file, do you think it should be moved as well? All the other files contain get and post functions for each of the pages?

sohilpandya commented 6 years ago

Keep require login file inside the controller. 👍