Skip to content

Conversation

@rrivem
Copy link
Contributor

@rrivem rrivem commented Aug 18, 2017

  • postgres with docker config.

@rrivem rrivem self-assigned this Aug 18, 2017
@rrivem rrivem changed the base branch from master to develop August 18, 2017 13:30
@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.683% when pulling 69670ac on feature/pg-setup into ab5a44e on develop.

docs/setup.md Outdated
* Install [Docker](https://docs.docker.com/engine/installation/) and [docker-compose](https://docs.docker.com/compose/install/)
* Run `yarn database:docker-server` to create the docker container for the SQL Server (volume to persist data will be mounted)

Once you have configured your SQL Server, run the following commands:
Copy link
Collaborator

Choose a reason for hiding this comment

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

update SQL Server text to PostgreSQL

docs/setup.md Outdated
* Create new database called *reactseeddb*. Use following credentials:
* user: `postgres`
* password: `Password.01`
* Update the `postgres` password in the `server/datastore/create/knexfile.js`
Copy link
Collaborator

Choose a reason for hiding this comment

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

new password is needed 'cause docker image config?

@@ -0,0 +1,3 @@
exports.seed = function(knex) {
return knex.raw('CREATE DATABASE reactseeddb');
Copy link
Collaborator

Choose a reason for hiding this comment

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

append the next two seeds in batch: kenx.raw(...).then( ... )

  • CREATE DATABASE
  • CREATE USER
  • GRANT

connection: {
host: 'localhost',
user: 'postgres',
password: 'sa.pg.01',
Copy link
Collaborator

Choose a reason for hiding this comment

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

update docs (setup.md) with initial db crendentials

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.683% when pulling f33720b on feature/pg-setup into ab5a44e on develop.

Copy link
Collaborator

@carloluis carloluis left a comment

Choose a reason for hiding this comment

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

🐳 👍

@rrivem rrivem merged commit a91103e into develop Aug 18, 2017
@rrivem rrivem deleted the feature/pg-setup branch August 18, 2017 15:51
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.

4 participants