TheLethalCode / Artemis-arrow

14 stars 31 forks source link

Database #32

Closed gargnishith closed 5 years ago

gargnishith commented 5 years ago

Here in this file i have created a new web page using flask. In conn.py i have established connection to sqlite3 database and created a table named as "user" which store the information of the user from the sigup page. In templates folder i have created HTML file for templates required during for the webpage.In main.py it has main script of flask through which whole app can run.

TheLethalCode commented 5 years ago

Dude, what even? You just changed the structure of the entire repo! You not only changed the main app.py file, you deleted the previous templates and statics and created your own. There are a lot of changes:- 1) The current front end is better than the one you are proposing. So leave the current front end as it is and link the back end with it. 2) No need to create a separate file just to create the database. See if the file exists, if it doesn't, this is the first time and create one. 3) Don't create a separate flask file. Edit all you want on the existing app.py. 4) Don't create a separate folder for the database. You just have to place the database file in the main repo. 5) Follow the PEP8 conventions These all are I got on a first glance. Rectifiy these first.