Skip to content

Issues: FACN1/week6-D-REEM-TEAM

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

21 Open 14 Closed
21 Open 14 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Assignee
Filter by who’s assigned
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Issues list

Local install instructions
#52 opened Mar 31, 2017 by m4v15
render function could be simplified
#50 opened Mar 31, 2017 by jwld
@edgarfinn @Karyum
connection ssl can be set inside the options object
#49 opened Mar 31, 2017 by jwld
db_build.sql page can be more short ! Addressed
#48 opened Mar 31, 2017 by hibaakel
@Karyum
1
Really long lines in database.js
#47 opened Mar 31, 2017 by emilyb7
Error handling in database.js
#46 opened Mar 31, 2017 by emilyb7
inconsistent single and double quotes in database.js
#45 opened Mar 31, 2017 by jwld
@RQ16
var and const Addressed
#43 opened Mar 31, 2017 by emilyb7
@RQ16
1
database.js potential hang
#41 opened Mar 31, 2017 by mattlub
1
database.js response sent with no status code
#40 opened Mar 31, 2017 by mattlub
DB table names duplicate
#39 opened Mar 31, 2017 by emilyb7
@RQ16
html input names
#38 opened Mar 31, 2017 by mattlub
might not be good practice putting scripts outside the body tag
#37 opened Mar 31, 2017 by jwld
unneccessary map in database.js
#36 opened Mar 31, 2017 by mattlub
1
use labels for input elements instead of spans Addressed
#35 opened Mar 31, 2017 by jwld
@edgarfinn
mock data in public folder could be stored in a separate tests folder Addressed
#34 opened Mar 31, 2017 by mattlub
@Karyum
listener.js could be removed Answered
#33 opened Mar 31, 2017 by mattlub
1
why the div within a div?
#32 opened Mar 31, 2017 by jwld
1
db table name
#30 opened Mar 31, 2017 by mattlub
@RQ16
restaurant id in ratings table should be foreign key
#29 opened Mar 31, 2017 by mattlub
README updated
#7 opened Mar 29, 2017 by Karyum
ProTip! Type g p on any issue or pull request to go back to the pull request listing page.