integrate user login system with database

Ishi01 / CITS5505Group41Project

CITS5505 Agile Web Develoment Group Project

MIT License

1 stars 0 forks source link

integrate user login system with database #14

Closed berry1104 closed 6 months ago

berry1104 commented 6 months ago

add new features:

user log in: verify name and password
user log out
user registration(below login form)
requiring user login( for now, only requiring in world map page)

berry1104 commented 6 months ago

other changes

add ER_diagram.md for the databse entity relationship diagram( for now, only two tables: users and games)
add DEVELOPMENT_GUIDE.md to record how to set up enviroment in details

stuar7 commented 6 months ago

Looks good! The instructions for the database initialisation are a bit incorrect since if the migrations file already exists the process to initialise the database is: flask db migrate -> flask db upgrade. The init function doesn't have to be run again by any user pulling the repository.

I'm also considering that individuals should be able to use the world map game without logging in, just their performance data won't be saved or tracked.

I updated the .gitignore file to not push the database in 346770c.

berry1104 commented 6 months ago

thanks for the reviews, i will make these changes:

remove the flask db init step
temporarily remove the "requiring login" feature, as we might discuss when data should be stored tomorrow, like add a new endpoint(/.game) for enter "game mode" where users are required login and data will be store