search

changeweb / Unifiedtransform

A school management Software
https://changeweb.github.io/Unifiedtransform/
GNU General Public License v3.0
2.8k stars 1.23k forks source link

I worked on the migration database integrity constraints #301

Open orajiakuchibuchi opened 3 years ago

orajiakuchibuchi commented 3 years ago

Screenshot from 2021-05-21 00-15-12 Screenshot from 2021-05-21 00-14-09

Before my contribution, I realized the system was working fine but there were no foreign key integrity constraints in the database design. I worked on the migration schemas, adding the foreign key integrity constraints. I also rearranged the order of the migrations. This provides a better database relationship and schema structure. Test run on a new database. After running the php artisan migrate command, Go to http://localhost/phpmyadmin Click the database > Database designer

Below are screenshots of the old and new database design view

orajiakuchibuchi commented 3 years ago

I worked on the master dashboard as well. adding the side menu, and dashboard card. new cards for the following: total number of schools, admins, teachers, and students. Added the side menu screencapture-127-0-0-1-8001-masters-2021-05-22-00_27_28

changeweb commented 3 years ago

@orajiakuchibuchi Great work! I didn't add foreign key constraint because in some cases, we need to delete a row from a table but it could not be deleted just because of foreign key constrained was integrated.

If you can add some Tests which test of delete actions without being affected by foreign key, I would be confident enough to merge the PR.