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.

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

build: release v5.4.0 #1493

Merged
merged 36 commits into from
Mar 30, 2021
Merged

build: release v5.4.0 #1493

merged 36 commits into from
Mar 30, 2021

Conversation

mantariksh
Copy link
Contributor

New

  • chore: take csv and attachments download out of beta #1466

Improved

  • refactor: remove unnecessary field validator JS tests in favour of TS tests #1475
  • feat: revert custom code for CorpPass cloud migration #1476
  • refactor: migrate field validator tests to typescript #1444
  • refactor: remove deprecated analytics API endpoints #1427
  • feat: remove adminform passThroughFeedback #1298
  • refactor: Migrate 3 more filters to handleEncryptedSubmission #1420
  • refactor: move Verification files together (part 1) #1450
  • build: fix mergify config #1456
  • feat: add samesite GA cookie property to enable GA in iframe #1463
  • chore: copy changes #1408
  • fix: shortened sms otp copy for accessibility #1461
  • ref: collapse middlewares of /adminform/submissions/download #1442
  • refactor: display verification error message if present #1443
  • feat: remove isTemplate validation on duplicate template api #1432
  • refactor: migrate table validator to typescript #1316

Dependency upgrades

  • chore(deps-dev): bump eslint from 7.22.0 to 7.23.0 #1473
  • chore(deps-dev): bump mockdate from 3.0.4 to 3.0.5 #1472
  • chore(deps-dev): bump @types/node from 14.14.35 to 14.14.37 #1471
  • chore(deps-dev): bump @types/jest from 26.0.21 to 26.0.22 #1470
  • fix(deps): bump aws-sdk from 2.871.0 to 2.874.0 #1468
  • chore(deps-dev): bump @babel/core from 7.13.10 to 7.13.13 #1467
  • fix(deps): bump twilio from 3.58.0 to 3.59.0 #1465
  • fix(deps): bump aws-sdk from 2.870.0 to 2.871.0 #1464
  • chore(deps-dev): bump @typescript-eslint/eslint-plugin #1448
  • chore(deps-dev): bump mockdate from 3.0.3 to 3.0.4 #1458
  • fix(deps): bump connect-mongo from 4.4.0 to 4.4.1 #1459
  • chore(deps-dev): bump testcafe from 1.12.0 to 1.13.0 #1460
  • chore(deps-dev): bump @babel/preset-env from 7.13.10 to 7.13.12 #1457
  • fix(deps): bump aws-sdk from 2.866.0 to 2.870.0 #1449
  • chore(deps-dev): bump husky from 5.1.3 to 5.2.0 #1447
  • chore(deps-dev): bump @typescript-eslint/parser from 4.18.0 to 4.19.0 #1433
  • fix(deps): bump @sentry/browser from 6.2.2 to 6.2.3 #1422
  • chore(deps-dev): bump sinon from 9.2.4 to 10.0.0 #1429

tshuli and others added 30 commits March 23, 2021 15:32
chore: merge release v5.3.0 back to develop
Bumps [sinon](https://github.com/sinonjs/sinon) from 9.2.4 to 10.0.0.
- [Release notes](https://github.com/sinonjs/sinon/releases)
- [Changelog](https://github.com/sinonjs/sinon/blob/master/CHANGELOG.md)
- [Commits](sinonjs/sinon@v9.2.4...v10.0.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: collapse middlewares for submission download endpoint

* feat: update JSDoc of handleStreamEncryptedResponses

now shows all possible http status codes the handler can return

* test(EncryptSubCtl): add tests for handleStreamEncryptedResponses
Bumps [@sentry/browser](https://github.com/getsentry/sentry-javascript) from 6.2.2 to 6.2.3.
- [Release notes](https://github.com/getsentry/sentry-javascript/releases)
- [Changelog](https://github.com/getsentry/sentry-javascript/blob/master/CHANGELOG.md)
- [Commits](getsentry/sentry-javascript@6.2.2...6.2.3)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [husky](https://github.com/typicode/husky) from 5.1.3 to 5.2.0.
- [Release notes](https://github.com/typicode/husky/releases)
- [Commits](typicode/husky@v5.1.3...v5.2.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.866.0 to 2.870.0.
- [Release notes](https://github.com/aws/aws-sdk-js/releases)
- [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md)
- [Commits](aws/aws-sdk-js@v2.866.0...v2.870.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* build: rename .mergify.yml to mergify.yml

* build: add approval to mergify steps
Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.13.10 to 7.13.12.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.13.12/packages/babel-preset-env)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [testcafe](https://github.com/DevExpress/testcafe) from 1.12.0 to 1.13.0.
- [Release notes](https://github.com/DevExpress/testcafe/releases)
- [Changelog](https://github.com/DevExpress/testcafe/blob/master/CHANGELOG.md)
- [Commits](DevExpress/testcafe@v1.12.0...v1.13.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [connect-mongo](https://github.com/jdesboeufs/connect-mongo) from 4.4.0 to 4.4.1.
- [Release notes](https://github.com/jdesboeufs/connect-mongo/releases)
- [Changelog](https://github.com/jdesboeufs/connect-mongo/blob/master/CHANGELOG.md)
- [Commits](jdesboeufs/connect-mongo@v4.4.0...v4.4.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [mockdate](https://github.com/boblauer/MockDate) from 3.0.3 to 3.0.4.
- [Release notes](https://github.com/boblauer/MockDate/releases)
- [Changelog](https://github.com/boblauer/MockDate/blob/master/CHANGELOG.md)
- [Commits](boblauer/MockDate@v3.0.3...v3.0.4)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: added currency to copy

* chore: change copy of inaccurate tooltip

* chore: make tooltips consistent

* refactor: remove superfluous ng-switch statement

* refactor: british spelling

Co-authored-by: Yuanruo Liang <yuanruo@open.gov.sg>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.870.0 to 2.871.0.
- [Release notes](https://github.com/aws/aws-sdk-js/releases)
- [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md)
- [Commits](aws/aws-sdk-js@v2.870.0...v2.871.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [twilio](https://github.com/twilio/twilio-node) from 3.58.0 to 3.59.0.
- [Release notes](https://github.com/twilio/twilio-node/releases)
- [Changelog](https://github.com/twilio/twilio-node/blob/main/CHANGES.md)
- [Commits](twilio/twilio-node@3.58.0...3.59.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* refactor: Migrate validateAndProcessEncryptSubmission, isSpcpAuthenticated, and verifyMyInfoVals to handleEncryptedSubmission

* attempt to fix encrypt-submission.routes.spec.ts test failure

* fix typescript spcp tests

* fix lints

* added TODOs per yuanruo's comments
Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.13.10 to 7.13.13.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.13.13/packages/babel-core)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.871.0 to 2.874.0.
- [Release notes](https://github.com/aws/aws-sdk-js/releases)
- [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md)
- [Commits](aws/aws-sdk-js@v2.871.0...v2.874.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [@types/jest](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jest) from 26.0.21 to 26.0.22.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jest)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) from 14.14.35 to 14.14.37.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [mockdate](https://github.com/boblauer/MockDate) from 3.0.4 to 3.0.5.
- [Release notes](https://github.com/boblauer/MockDate/releases)
- [Changelog](https://github.com/boblauer/MockDate/blob/master/CHANGELOG.md)
- [Commits](boblauer/MockDate@v3.0.4...v3.0.5)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [eslint](https://github.com/eslint/eslint) from 7.22.0 to 7.23.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v7.22.0...v7.23.0)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: remove passThroughFeedback

* feat: remove traces of POST /adminform/feedback

* fix: remove unused var "feedback"

* fix: remove unused var "resourceUrl"

* fix: revert deletion of postFeedback and check vm.isPreview

* fix: remove vm.isPreview check and modify postFeedback instead

* chore: run linter

* refactor: add else-clause to isPreview check and remove duplicate return statement
r00dgirl and others added 6 commits March 29, 2021 10:14
* chore: remove beta-icon

* chore: remove beta-icon

* chore: remove beta-icon
Co-authored-by: Yuanruo Liang <yuanruo@open.gov.sg>
Tests for the following field validators were ported:

- date
- dropdown
- decimal
- home number
- nric
- number
- radio
- rating
- text
… tests (#1475)

Co-authored-by: Yuanruo Liang <yuanruo@open.gov.sg>
@mantariksh
Copy link
Contributor Author

mantariksh commented Mar 30, 2021

@karrui

Tests

CorpPass cloud

  • Create, activate and submit CorpPass form

@mantariksh
Copy link
Contributor Author

mantariksh commented Mar 30, 2021

@karrui

Tests

Copy changes

  • Email confirmation tooltip in edit field modal shows "Customise an email acknowledgement to respondents"

@mantariksh
Copy link
Contributor Author

mantariksh commented Mar 30, 2021

@karrui

Tests

Storage mode middleware refactors

  • Downloading CSV + attachments works as expected on Chrome

  • Current bug discovered that is already in prod: exporting csv then exporting again will result in a permanent spinner with no downloads being completed

@mantariksh
Copy link
Contributor Author

mantariksh commented Mar 30, 2021

@seaerchin

Tests

Table validator

  • Create two required table fields. Hide one of them behind logic. Check that normal submission using frontend succeeds for both visible and hidden field.
  • Check that following validation works when submitting programatically
  • Answer cannot be empty for required column
  • For dropdown column, invalid answer (e.g. 'abc') is rejected
  • Answer cannot have more than max rows if addMoreRows is true
  • Answer cannot have fewer than min rows

@mantariksh
Copy link
Contributor Author

mantariksh commented Mar 30, 2021

@mantariksh

Tests

Intranet IE11 runthrough

  • Create email mode form with all fields and custom start/end page. Activate and submit form. Check that you can receive responses in your intranet inbox.
  • Collaborators can be added in email mode.
  • Create storage mode form with all fields and custom start/end page. Activate and submit form.
  • Response CSV can be downloaded for storage mode.
  • Responses can be viewed in Data tab in storage mode.
  • Attachments for individual fields can be downloaded from Data tab in storage mode.
  • Attachment ZIP for submission can be downloaded from Data tab in storage mode.
  • Submission IDs can be filtered in storage mode.
  • Collaborators can be added and their collaborating rights edited in storage mode.

@mantariksh mantariksh merged commit 3ab46d7 into release Mar 30, 2021
@karrui karrui deleted the release-v5.4.0 branch April 21, 2021 02:17
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.

8 participants