senaduka/old-web
1
0
Fork 0
Code Issues Pull Requests 5 Actions Packages Projects Releases Wiki Activity

Change migrations - use string instead of enum #7

Merged
bilal.catic merged 2 commits from change-migrations-use-string-instead-of-enums into master 2019-05-24 15:51:01 +02:00
Conversation 6 Commits 2 Files Changed 5 +8 -30
bilal.catic commented 2019-05-20 19:34:33 +02:00 (Migrated from gitlab.com)
Copy Link

SMOKE TEST :

  1. Stop your docker container with database (docker stop pg_marketalerts)
  2. Remove docker container (docker rm pg_marketalerts)
  3. From project directory run setup script (npm run setup)
  4. Run project (npm start or npm run start-mon)
  5. Test app, proceed through wizard steps
  6. Open database, check that selected values are saved correctly

¯\＿(ツ)＿/¯

SMOKE TEST : 1. Stop your docker container with database (`docker stop pg_marketalerts`) 2. Remove docker container (`docker rm pg_marketalerts`) 3. From project directory run setup script (`npm run setup`) 4. Run project (`npm start` or `npm run start-mon`) 5. Test app, proceed through wizard steps 6. Open database, check that selected values are saved correctly ¯\＿(ツ)＿/¯
bilal.catic commented 2019-05-21 11:48:36 +02:00 (Migrated from gitlab.com)
Copy Link

unmarked as a Work In Progress

unmarked as a **Work In Progress**
bilal.catic commented 2019-05-21 11:52:14 +02:00 (Migrated from gitlab.com)
Copy Link

changed the description

changed the description
bilal.catic commented 2019-05-21 15:31:43 +02:00 (Migrated from gitlab.com)
Copy Link

added 4 commits

  • de5bbe1e...315a2974 - 2 commits from branch master
  • 58ae4305 - modify migrations - use string instead of enum
  • 126da488 - modify realEstateRequest model to use String instead of enum

Compare with previous version

added 4 commits <ul><li>de5bbe1e...315a2974 - 2 commits from branch <code>master</code></li><li>58ae4305 - modify migrations - use string instead of enum</li><li>126da488 - modify realEstateRequest model to use String instead of enum</li></ul> [Compare with previous version](https://gitlab.com/saburly/marketalarm/web/merge_requests/7/diffs?diff_id=42722803&start_sha=de5bbe1ebb1ebf15b8c613994a4e88597547afc9)
bilal.catic commented 2019-05-21 15:41:17 +02:00 (Migrated from gitlab.com)
Copy Link

mentioned in merge request !8

mentioned in merge request !8
nedimu commented 2019-05-24 15:51:01 +02:00 (Migrated from gitlab.com)
Copy Link

merged

merged
nedimu commented 2019-05-24 15:51:02 +02:00 (Migrated from gitlab.com)
Copy Link

mentioned in commit 89a3c9e355

mentioned in commit 89a3c9e355a8f17b179984df48aac685edf8455d
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
senaduka
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: senaduka/old-web#7
Delete Branch "change-migrations-use-string-instead-of-enums"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?