This repository has been archived by the owner on May 7, 2021. It is now read-only.
Testing a run of our pipeline build to prod for a quick smoke test #1558
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* change all back button copy * Update Privacy and Consent Page content
* change word order to make it more scannable * english content update * french content updates
* updates to money page content * update information page content * devices page copyedit * update content on business page * what happened page content edits
* clear the data in the previous report when reporter wants to another report by clicking on "create a new report" in the thank you page * deleted unused items Please enter the commit message for your changes. Lines starting * deleted unwanted line Please enter the commit message for your changes. Lines starting
* changed fields on contact page * linted
* started working on the feedback thing * implemented the mid service feedback component * added a test, removed typo in file name
* content edits * french * remove unused tag
* Update location page content for launch version * Update file upload page copy * remove unused P tag Co-authored-by: ngosset <ngosset@gmail.com>
Co-authored-by: ngosset <ngosset@gmail.com>
* Add Terms and conditions page * remove unused variable
* Added new and file uploading related text . * Rearranged text to match new figma look * Added file upload type, extensions restrictions. * Added client side file restrictions, updated page.
Co-authored-by: ngosset <ngosset@gmail.com>
* Minor tweaks to contact page content * Punctuation and microcopy on feedback component mid-flow * add space after conditional text in thank you page * change conditional text about email to not include summary of report * clarify what happens next on thank you page * Fixes on confirmation page content * Update device content * missing translations * add new string to Device.js file * change description to match change * add missing translations * fix broken test
* Saving file as blob to mongodb * Moved to Azure Blob Storage * Added to example env * LGTM errors * Merged with avscanner code, storing scanner result in json... refactor - ASYNC ALL THE THINGS * Added clam url to env example * Refactoring some more * Missed part of the report ID on server stuff * look at this steve
* update review page * fixes #1410 * change feedback button copy EN
…nt edits (#1420) * adding tip on how did it start page * add alert on money page * add tips to impact pages and fix next buttons in french
* Remove form from full privacy statement * Add all links to the Full Privacy Statement * fix failing test * missing translation * fix email links * remove back button to resolve test * reformatting
* Blob Storage and App Variables Here I am adding all of the required app service environment variables and adding in the blob storage and configuring it. * Implementing review comments * removing unencrypted reports
* Changing names back changing resource names back * add uid variables
* Added basic render test for HowDidItStartForm.
Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com>
Co-authored-by: dhcdn <51758091+davidheerema@users.noreply.github.com> Co-authored-by: DianeLiu2019 <51336707+DianeLiu2019@users.noreply.github.com>
* [a11y] aria landmark - fix * [a11y] added the missing form label * Updated the color tone slightly to meet WCAG requirement * Adding a darker shade for the active state * Added in another PR
* Fix command typo * Add sleep and comments * fix db name * updaing blob name * string escape changes * metrics typo
* add app info-best viewed with Chrome or Firefox on landing page * update copy in alert and only have one alert per page * edit string Co-authored-by: Anik Brazeau <38330843+anikbrazeau@users.noreply.github.com>
…ge "I was contacted other" (#1495)
…1542) * Removed the Landing page side by side containers * Removed unused packages * Removed unused package * removed call button, fixed spacing between things * linted Co-authored-by: amazingphilippe <amazingphilippe@gmail.com>
…1542) * Removed the Landing page side by side containers * Removed unused packages * Removed unused package * removed call button, fixed spacing between things * linted Co-authored-by: amazingphilippe <amazingphilippe@gmail.com>
Co-authored-by: ngosset <ngosset@gmail.com>
|
This pull request fixes 1 alert when merging d150ab2 into 7bb8b33 - view on LGTM.com fixed alerts:
|
davidheerema
approved these changes
Feb 25, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Testing our deployment to prod environment by doing a PR from master.