Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

[FEATURE] Adding support for PostgreSQL #55

Merged
merged 14 commits into from
Jul 20, 2019
Merged

[FEATURE] Adding support for PostgreSQL #55

merged 14 commits into from
Jul 20, 2019

Conversation

M4ST3RX
Copy link
Collaborator

@M4ST3RX M4ST3RX commented Jul 20, 2019

Issue or feature explanation

This feature adds support for PostgreSQL database connections. See #46

Proposed solution/change

See commits for the changes. Few important bits:

  • Created Prequel facade as a universal query builder
  • Changed usage of DB to custom PDB facade
  • Added an argument to DatabaseConnector::getConnection so PostgreSQL can query multiple databases.
  • Added getTableData function to retrieve the rows of a table.

@QuintenJustus QuintenJustus self-requested a review July 20, 2019 20:45
@QuintenJustus QuintenJustus added the FEATURE New feature or request label Jul 20, 2019
@QuintenJustus
Copy link
Member

Looks good!

@QuintenJustus QuintenJustus merged commit 437c2d6 into Protoqol:Dev Jul 20, 2019
@James-N-M
Copy link
Collaborator

@M4ST3RX next time you take on a issue like this lets pair it, I've love to learn how you went about it 💯

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FEATURE New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants