Skip to content

PostgreSQL -> MS SQL change (ported from PA Tool)

J Michael Cherry requested to merge dev/wl/mssql-port into master

Description

  • PostgreSQL -> MS SQL change (ported from PA Tool)

Accompanying documentation was also copied over (see the MS SQL Transfer.md file for the full details of the implementation of said change)

  • Also fixed a small error with the verifyJWT() function in jwtUtil.js
  • Also added a convenient command: ./scripts.sh sql to log into the MS SQL Container with 1 short command (as opposed to 2 long ones)

JIRA Ticket Number

N/A

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update
  • Dependency change

Screenshots

MS SQL DB tests (adding 2 entries to the DB via the API button on the home page & then running a simply SQL Select statement) MS_SQL_database_test

Yarn tests passing Yarn_tests_passing

Testing

Unit tests:

  1. docker-compose build
  2. docker-compose up
  3. ./scripts.sh test

Testing Database:

  1. Go to the home page (http://localhost:81/home)
  2. Click on the "POST to api/sample-endpoint" button a few times
  3. ./scripts.sh sql
  4. You should be in the MS SQL Container's SQL terminal now. Submit this command: SELECT * from custom_models_sample_model
  5. Then submit this command: GO

Checklist

  • I have added or updating documentation/one pagers related to my code changes

  • I have commented my code, particularly in hard-to-understand areas

  • My changes generate no new compiler warnings

- [ ] I have added tests that prove my fix is effective or that my feature works

- [ ] I have researched WCAG2.1 accessibility standards and met them in this PR (can be navigated using a keyboard)

- [ ] My changes look good on IE 11+ and Chrome

- [ ] My changes look good on various screen sizes (mobile, square screen)

- [ ] I have translated new text or created a JIRA to do it

Edited by J Michael Cherry

Merge request reports

Loading