Skip to content

Tidying Up Data Layer#6

Merged
ThomasCode92 merged 6 commits intomainfrom
tidying-up-data-layer
Jun 7, 2025
Merged

Tidying Up Data Layer#6
ThomasCode92 merged 6 commits intomainfrom
tidying-up-data-layer

Conversation

@ThomasCode92
Copy link
Owner

@ThomasCode92 ThomasCode92 commented Jun 6, 2025

🗃️ Organize Data Access Layer

✨ Summary

This PR refactors the backend architecture by separating SQL queries from the application logic. All queries have been moved to a dedicated file within the server/db directory, resulting in cleaner, more maintainable code and a clear separation of concerns.


🛠️ What’s Included

  • 📦 Data Access Layer Cleanup
    • Centralizes all SQL queries in a single module: server/db/queries.ts.
    • Promotes reusability and consistency in how data is accessed.
  • 🧼 Component Cleanup
    • Removes direct database logic from React components s.
    • Ensures UI code is focused solely on rendering and user interaction.

📌 Notes

  • This refactor improves code readability and scalability.
  • Future database changes can now be made in one place without touching business or UI logic.

@netlify
Copy link

netlify bot commented Jun 6, 2025

Deploy Preview for t4s-drive-tutorial ready!

Name Link
🔨 Latest commit ad47e2a
🔍 Latest deploy log https://app.netlify.com/projects/t4s-drive-tutorial/deploys/68428c82d42dfd0008c12372
😎 Deploy Preview https://deploy-preview-6--t4s-drive-tutorial.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ThomasCode92 ThomasCode92 changed the title Tidying up data layer Tidying up Data Layer Jun 7, 2025
@ThomasCode92 ThomasCode92 changed the title Tidying up Data Layer Tidying Up Data Layer Jun 7, 2025
@ThomasCode92 ThomasCode92 merged commit 03fb544 into main Jun 7, 2025
5 checks passed
@ThomasCode92 ThomasCode92 deleted the tidying-up-data-layer branch June 7, 2025 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant