Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Connect database with the frontend app #95

Merged
merged 3 commits into from
May 10, 2021
Merged

Connect database with the frontend app #95

merged 3 commits into from
May 10, 2021

Conversation

kam56
Copy link
Contributor

@kam56 kam56 commented May 10, 2021

Completed:

PostgreSQL chosen as DBMS
The related client installed on Electron js
Related connection is configured into the config file

Next:
Persist all the mission, settings, and user registration data into the database

Closes #89

kam56 and others added 3 commits May 8, 2021 13:48
@kam56
Connect database system to the frontend
6079b96 
* Install database system
* Install related client

* Test connection by running db_connection.js

Closes #89
@kam56
Retrieve client data from confiq folder
c9ee09e 
* Retrieve data from config json file

Fixes #89
@kam56
Merge pull request #94 from DIT112-V21/master
bb68023 
Connect the frontend with the database
@kam56 kam56 added sprint-3 User stories & bugs for TBD on sprint 3 test Issue, item or user story needs to be tested JS Javascript work NodeJS NodeJS and ElectronJS related tasks M3 Milestone 3 related issues labels May 10, 2021
@kam56 kam56 added this to the Mission control section milestone May 10, 2021
@kam56 kam56 self-assigned this May 10, 2021
@kam56 kam56 added the database Issues related to database and interactions with database label May 10, 2021
OneMoreOreo
OneMoreOreo approved these changes May 10, 2021
View reviewed changes
Copy link
Contributor

@OneMoreOreo OneMoreOreo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Work! Very useful part for the missions.

@OneMoreOreo OneMoreOreo merged commit 4c1db62 into master May 10, 2021
@kam56 kam56 deleted the db-connection branch May 11, 2021 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OneMoreOreo OneMoreOreo OneMoreOreo approved these changes

@axe007 axe007 Awaiting requested review from axe007

@nikalc nikalc Awaiting requested review from nikalc

@ramadi-a ramadi-a Awaiting requested review from ramadi-a

Assignees

@kam56 kam56

Labels
database Issues related to database and interactions with database JS Javascript work M3 Milestone 3 related issues NodeJS NodeJS and ElectronJS related tasks sprint-3 User stories & bugs for TBD on sprint 3 test Issue, item or user story needs to be tested
Projects
None yet
Milestone
Mission control section
Development

Successfully merging this pull request may close these issues.

[Task] Install database system and connect to Frontend app
2 participants
@kam56 @OneMoreOreo