Start using migrations #5

Merged
bilal.catic merged 7 commits from start-using-migrations into development 2018-06-26 16:38:58 +02:00
bilal.catic commented 2018-06-26 13:50:18 +02:00 (Migrated from gitlab.com)

New : use migration instead of changing sql dump

Task : add first name, last name and mail(email) to the order

Smoke test :

open Co-Market and select first item on the left (or some other that is working ; example url : http://localhost:8000/co-market/14/302), add it to the cart. Go to the cart and complete ordering process. When finished, check in database that order contains required user details following this steps :

in terminal, run :
sudo docker exec -it legacywiaas_db_1 /bin/bash

now, open mysql : mysql -u root -p (password is root)

execute in mysql : use ricoh_dash
and after that : SELECT billingFirstName, billingLastName, billingMail from orders;

all data should be empty except in one row, that is order made by you

New : use migration instead of changing sql dump Task : add first name, last name and mail(email) to the order Smoke test : open Co-Market and select first item on the left (or some other that is working ; example url : http://localhost:8000/co-market/14/302), add it to the cart. Go to the cart and complete ordering process. When finished, check in database that order contains required user details following this steps : in terminal, run : sudo docker exec -it legacywiaas_db_1 /bin/bash now, open mysql : mysql -u root -p (password is root) execute in mysql : use ricoh_dash and after that : SELECT billingFirstName, billingLastName, billingMail from orders; all data should be empty except in one row, that is order made by you
bilal.catic commented 2018-06-26 13:58:15 +02:00 (Migrated from gitlab.com)

added 1 commit

Compare with previous version

added 1 commit <ul><li>bea9ba5e - restore URL-s to live server</li></ul> [Compare with previous version](https://gitlab.com/saburly/wiaas/wiaas-legacy/merge_requests/5/diffs?diff_id=18699658&start_sha=9b26ff1cd46509172ca44b344d7c3d004b1a0465)
bilal.catic commented 2018-06-26 13:58:44 +02:00 (Migrated from gitlab.com)

changed the description

changed the description
nedimu commented 2018-06-26 15:49:48 +02:00 (Migrated from gitlab.com)

Why do you repeat the values of environment variables, like DB name , user and pass if we are using the same credentials.

This is ok only if you are planning to add diffenet DB user to phinx

Why do you repeat the values of environment variables, like DB name , user and pass if we are using the same credentials. This is ok only if you are planning to add diffenet DB user to phinx
bilal.catic commented 2018-06-26 15:57:08 +02:00 (Migrated from gitlab.com)

I left PROD and TEST variables in case we need different databases, but they are obsolete, should I remove those ?

I left PROD and TEST variables in case we need different databases, but they are obsolete, should I remove those ?
nedimu commented 2018-06-26 15:57:58 +02:00 (Migrated from gitlab.com)

This can be written better.

You don't need to repeat keys like host , name etc... What if we had more environments ??
Just determine the environment first , load values for that environment into variables
and add variable values to those keys

This can be written better. You don't need to repeat keys like host , name etc... What if we had more environments ?? Just determine the environment first , load values for that environment into variables and add variable values to those keys
nedimu commented 2018-06-26 16:38:58 +02:00 (Migrated from gitlab.com)

merged

merged
nedimu commented 2018-06-26 16:39:00 +02:00 (Migrated from gitlab.com)

mentioned in commit 92f237c82f

mentioned in commit 92f237c82f3f28232feff7f2592d56883427ea66
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: senaduka/old-wiaas-legacy#5