mutheujoy / MISSING-PERSONS-DATABASE-2024-KENYA-FINANCE-BILL-PROTESTS-

AI-powered platform for tracking missing Kenyans during the 2024 #RejectFinanceBill2024 protests in Kenya
Other
87 stars 50 forks source link

chore: Update and Refactor Flask Application and Update Database Configuration #26

Open Dovineowuor opened 1 month ago

Dovineowuor commented 1 month ago

Refactor Flask Application and Update Database Configuration

Description: This pull request includes several key updates and refactoring changes aimed at improving the codebase's readability, security, and maintainability. The changes are detailed below:

Commits:

  1. Refactor Flask Application and Update Database Configuration:

    • Improved modularization of the Flask application for better code organization and readability.
    • Enhanced error handling in database operations to prevent application crashes and ensure data integrity.
    • Updated logging for better traceability and debugging support.
    • Improved security by utilizing Flask's URL generation and avoiding manual URL construction.
    • Optimized logic to remove redundant conditions and improve performance.

    Commit ID: 6c83419, 058745e

  2. Update Footer Link to Privacy Policy:

    • Updated the footer link to the Privacy Policy in static/footer-copy.html to ensure it points to the correct URL.

    Commit ID: 43deca2

  3. Refactor HTML Structure and Update Page Title:

    • Refactored the HTML structure to enhance semantic layout and ensure compliance with modern web standards.
    • Updated page titles for better SEO and user understanding.

    Commit ID: ac7e876

Testing:

Notes:

Reviewer Instructions:

Screenshots: (if applicable)

Closing Issues: (if applicable)


mutheujoy commented 1 month ago

@bdigismat check on this

Dovineowuor commented 1 month ago

Seems @bdigismat ain't up to it, please do update to help me update my fork effectively

Dovineowuor commented 1 month ago

@mutheujoy

apexDev37 commented 1 month ago

Hey @Dovineowuor, thanks for the PR!

apexDev37 commented 1 month ago

In the future, I propose trying to avoid long-lived branches that progress parallel to main. This way integration is easier, other contributors can pull in your patches/updates into their checkouts, and you can get faster feedback from a review

apexDev37 commented 1 month ago

With that said, I'm happy to go through your changes🙂