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.18.0 #2282

Merged
merged 20 commits into from
Jul 1, 2021
Merged

build: release v5.18.0 #2282

merged 20 commits into from
Jul 1, 2021

Conversation

karrui
Copy link
Contributor

@karrui karrui commented Jul 1, 2021

Release v5.18.0

New

  • feat: rename "Reference Number" to "Response ID" #2277
  • feat: Admin form UI changes to "Edit Welcome" fields #2258

Fixes

  • fix: allow creation of storage form when emails are invalid #2263
  • fix: correctly retrieve targetFormId for redirect state #2261

Improvements

  • refactor: ensure consistent filesize const #2079

Dependency changes

  • fix: upgrade mongoose from 5.12.12 to 5.12.13 #2257
  • fix(deps): bump @sentry/browser from 6.7.2 to 6.8.0 #2269
  • fix(deps): bump @sentry/integrations from 6.7.2 to 6.8.0 #2268
  • fix(deps): bump aws-sdk from 2.932.0 to 2.933.0 #2252
  • fix(deps): bump aws-sdk from 2.933.0 to 2.935.0 #2259
  • fix(deps): bump aws-sdk from 2.935.0 to 2.936.0 #2273

Dev dependencies

  • chore(deps-dev): bump @typescript-eslint/eslint-plugin #2271
  • chore(deps-dev): bump @typescript-eslint/parser from 4.28.0 to 4.28.1 #2270
  • chore(deps-dev): bump core-js from 3.15.1 to 3.15.2 #2272
  • chore(deps-dev): bump coveralls from 3.1.0 to 3.1.1 #2274
  • chore(deps-dev): bump optimize-css-assets-webpack-plugin #2251
  • chore(deps-dev): bump prettier from 2.3.1 to 2.3.2 #2260
  • chore(deps-dev): bump ts-node-dev from 1.1.6 to 1.1.7 #2275

dependabot bot and others added 20 commits June 24, 2021 17:21
Bumps [optimize-css-assets-webpack-plugin](https://github.com/NMFR/optimize-css-assets-webpack-plugin) from 5.0.3 to 5.0.8.
- [Release notes](https://github.com/NMFR/optimize-css-assets-webpack-plugin/releases)
- [Commits](NMFR/optimize-css-assets-webpack-plugin@v5.0.3...v5.0.8)

---
updated-dependencies:
- dependency-name: optimize-css-assets-webpack-plugin
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.932.0 to 2.933.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.932.0...v2.933.0)

---
updated-dependencies:
- dependency-name: aws-sdk
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [prettier](https://github.com/prettier/prettier) from 2.3.1 to 2.3.2.
- [Release notes](https://github.com/prettier/prettier/releases)
- [Changelog](https://github.com/prettier/prettier/blob/main/CHANGELOG.md)
- [Commits](prettier/prettier@2.3.1...2.3.2)

---
updated-dependencies:
- dependency-name: prettier
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.933.0 to 2.935.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.933.0...v2.935.0)

---
updated-dependencies:
- dependency-name: aws-sdk
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: in "Edit Welcome", move "Color Theme" to btwn "Logo" and "Time taken for..." and add rows="12" to "Instructions..." textarea

* chore: run formatter
Bumps [@sentry/integrations](https://github.com/getsentry/sentry-javascript) from 6.7.2 to 6.8.0.
- [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.7.2...6.8.0)

---
updated-dependencies:
- dependency-name: "@sentry/integrations"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…#2270)

Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 4.28.0 to 4.28.1.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.28.1/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [@sentry/browser](https://github.com/getsentry/sentry-javascript) from 6.7.2 to 6.8.0.
- [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.7.2...6.8.0)

---
updated-dependencies:
- dependency-name: "@sentry/browser"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
this fixes the bug where the targetFormId was incorrectly retrieving the first 24 characters of the URL, which will fail as the API URLs have changed to be more semantic.

instead, take in the targetFormId directly as we already know the formId.
Bumps [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) from 4.28.0 to 4.28.1.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v4.28.1/packages/eslint-plugin)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* refactor: move MB to shared/constants and use MB throughout app

* ref: lint attachmentValidator and use relative import

Co-authored-by: Kar Rui Lau <[email protected]>
Bumps [core-js](https://github.com/zloirock/core-js/tree/HEAD/packages/core-js) from 3.15.1 to 3.15.2.
- [Release notes](https://github.com/zloirock/core-js/releases)
- [Changelog](https://github.com/zloirock/core-js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/zloirock/core-js/commits/v3.15.2/packages/core-js)

---
updated-dependencies:
- dependency-name: core-js
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.935.0 to 2.936.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.935.0...v2.936.0)

---
updated-dependencies:
- dependency-name: aws-sdk
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [coveralls](https://github.com/nickmerwin/node-coveralls) from 3.1.0 to 3.1.1.
- [Release notes](https://github.com/nickmerwin/node-coveralls/releases)
- [Commits](nickmerwin/node-coveralls@v3.1.0...3.1.1)

---
updated-dependencies:
- dependency-name: coveralls
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [ts-node-dev](https://github.com/whitecolor/ts-node-dev) from 1.1.6 to 1.1.7.
- [Release notes](https://github.com/whitecolor/ts-node-dev/releases)
- [Changelog](https://github.com/wclr/ts-node-dev/blob/master/CHANGELOG.md)
- [Commits](wclr/ts-node-dev@v1.1.6...v1.1.7)

---
updated-dependencies:
- dependency-name: ts-node-dev
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* refactor(emails-input): use ng-messages

* fix: fail validation only if responseMode is email

* fix: allow empty array and string for emails key in storage mode
* feat: update response email subject to use #

* feat: update response email body to use Response ID

* feat: update storage mode Data tab to use Response ID

* feat: update storage mode CSV to use Response ID

* feat: update email confirmation PDF to use Response ID

* build: empty commit to trigger build
chore: merge release v5.17.0 back to develop
@karrui
Copy link
Contributor Author

karrui commented Jul 1, 2021

@seaerchin

Tests

Response ID

  • Submit an email form. The email response email subject should look like formsg-auto: <formtitle> (#<submissionId>)(note the pound sign)
  • the email body should refer to "Response ID" instead of reference number
  • Submit a storage mode form. The data tab header when viewing the response should be "Response ID"
  • Download the storage mode CSV. The relevant header should be "Response ID"
  • The update email confirmation PDF to form filler should use "Response ID"

Q: when we download as zip, should it be RefNo or ResponseId
A: not urgent to fix, change to Response Id

@karrui
Copy link
Contributor Author

karrui commented Jul 1, 2021

@karrui

Tests

Admin form edit start page UI change

@karrui
Copy link
Contributor Author

karrui commented Jul 1, 2021

@yong-jie

Tests

Fix login redirect URL

Pre:

  • Grab an URL of a form you own, and another URL for a form you don't own (probably can be rubbish formId)
    Test:
  • Clear cookies. Go to the URL of the form you own again. Should be kicked out to the login page. Sign in should redirect you correctly back into the view admin form page of that form.
  • Clear cookies. Go to the URL of the form you don't own. Should be kicked out to an 404 error page, with a correct target form ID this time, e.g.

@karrui
Copy link
Contributor Author

karrui commented Jul 1, 2021

@karrui

Tests

Allow creation of storage mode form when emails are invalid (when toggling to email mode and toggling back)

  • Go to the create form modal and select "Email mode". Make the email list input invalid, e.g. by deleting all content in the input. Switch back to "Storage mode". You should still be able to create the form as long as the form title is valid. Make sure to go all the way through and create the form.
  • Select "Storage mode" and type an invalid form title. You should be prevented from creating the form.
  • Select "Email mode" and ensure that you are prevented from creating the form with the following types of inputs for the email list, with the correct error message in each case:
    • No input
    • Any invalid email present
    • Duplicate emails present
    • More than 30 non-duplicate emails present
  • Repeat previous test for email list in the Settings tab for an email mode form. If the input is invalid, it should revert to its previous value on blur.

mantariksh
mantariksh previously approved these changes Jul 1, 2021
@karrui karrui changed the base branch from develop to release July 1, 2021 10:07
@karrui karrui dismissed mantariksh’s stale review July 1, 2021 10:07

The base branch was changed.

@karrui karrui requested a review from mantariksh July 1, 2021 10:07
@karrui karrui merged commit 73cfc4a into release Jul 1, 2021
@karrui karrui deleted the release-v5.18.0 branch August 17, 2021 02:45
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