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

Merging staging into prod #4027

Merged
merged 76 commits into from
Oct 1, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
76 commits
Select commit Hold shift + click to select a range
7d957dd
10054 - removing blue banners and re-order
codyseibert Sep 14, 2023
0727ef2
10054 - adding interactor for generating pdf
codyseibert Sep 14, 2023
cf3db2c
10054: WIP updated arguments for generating entry of appearance, fram…
TomElliottFlexion Sep 14, 2023
7a1f530
10054: flesh out flow for returning url of generated entry of appearance
TomElliottFlexion Sep 15, 2023
ad6e12c
10054: WIP return url of generated entry of appearance and ATTEMPT to…
TomElliottFlexion Sep 15, 2023
15b63a9
10054 - work on the appearance pdf
TomElliottFlexion Sep 15, 2023
6f3abc4
10054: add redaction acknowledge checkbox/logic to entry of appear fi…
TomElliottFlexion Sep 15, 2023
384e63d
10054 - getting previews working in review page
TomElliottFlexion Sep 15, 2023
106570c
10054 - setting autogenerated document for entry of appearance if use…
TomElliottFlexion Sep 15, 2023
0327aa2
10054: WIP writing tests for newly created entry of appearance creati…
TomElliottFlexion Sep 15, 2023
95c6189
10054: add 'Respondent' to auto-generated EA documents for IRS
TomElliottFlexion Sep 18, 2023
10fcd07
10054: add tests for setDefaultGenerationTypeAction
TomElliottFlexion Sep 18, 2023
fac3050
10054: WIP: testing generateEntryOfAppearancePdfInteractor
TomElliottFlexion Sep 18, 2023
666df3c
10054 - working on coverage
codyseibert Sep 18, 2023
fcedac4
Merge branch '10054-generate-ea-documents' of github.com:flexion/ef-c…
codyseibert Sep 18, 2023
086985b
10054 - file document helper
codyseibert Sep 18, 2023
5d26219
10054: finish generateEntryOfAppearancePdfInteractor test
TomElliottFlexion Sep 18, 2023
ec2b0dc
10054 - adding more coverage
codyseibert Sep 18, 2023
ce3615a
10054: add visual test for entry of appearance pdf
TomElliottFlexion Sep 18, 2023
32ee2e0
Merge branch '10054-generate-ea-documents' of https://github.com/flex…
TomElliottFlexion Sep 18, 2023
59088d7
fixing failing unit test
codyseibert Sep 19, 2023
a53e1f9
10054 - fixing issue with integration test and consolidate group
codyseibert Sep 19, 2023
54a8538
10054 - fixing unit test
codyseibert Sep 19, 2023
b309417
10054 - fixing test
codyseibert Sep 19, 2023
9376a30
dep-updates: patch updates
ujahio Sep 20, 2023
c81162a
dep-udpates: aws minor updates
ujahio Sep 20, 2023
a567e80
10054 - reviewing pr with zach
codyseibert Sep 20, 2023
51d6581
10054: Remove hardcoded constants
Sep 20, 2023
00afcc7
Merge branch '10054-generate-ea-documents' of github.com:flexion/ef-c…
Sep 20, 2023
8bd7840
dep-updates: minor version update (5.17.1) to aws provider
ujahio Sep 20, 2023
fc07cc9
10054: Refactor allowExternalConsolidatedGroupFiling to be a computed…
Sep 20, 2023
22429aa
dep-updates: more minor updates
ujahio Sep 20, 2023
3f03ce3
10054: Add new integration test for practitioner requesting access to…
Sep 20, 2023
819a0bf
10054: Update unit tests
Sep 20, 2023
d43abbe
Merge branch 'staging' of https://github.com/ustaxcourt/ef-cms into d…
ujahio Sep 21, 2023
7e68db1
dep-updates: downgraded aws provider
ujahio Sep 21, 2023
5fdfbbf
dep-updates: esbuild update + last patch updates
ujahio Sep 21, 2023
76a3c0e
dep-updates: minor updates to jest related packages
ujahio Sep 21, 2023
6fec597
Merge branch 'staging' of https://github.com/ustaxcourt/ef-cms into d…
ujahio Sep 21, 2023
26a698a
dep-updates: last batch of minor updates
ujahio Sep 21, 2023
8541a65
dep-updates: major update of cypress
ujahio Sep 21, 2023
a02ffe0
dep-updates: improve doc instruction
ujahio Sep 21, 2023
f62fd2e
10054: revert allowExternalConsolidatedGroupFilingHelper to allowExte…
TomElliottFlexion Sep 22, 2023
8853a8f
10054: missed constant from state
TomElliottFlexion Sep 22, 2023
cfdd57b
Merge branch 'staging' of https://github.com/ustaxcourt/ef-cms into d…
ujahio Sep 22, 2023
ede7e1f
10054: reset external consolidated state values when switching back a…
TomElliottFlexion Sep 22, 2023
f571c83
dep-updates: aws cli update + publish new ecr image
ujahio Sep 22, 2023
b3559d4
dep-updates: docs cleanup
ujahio Sep 22, 2023
29450fd
dep-updates: update aws provider versions
ujahio Sep 22, 2023
2ef1f13
dep-updates: correct version of aws provider
ujahio Sep 22, 2023
050ceed
10054: use computed value by calling it instead of getting from cereb…
Sep 22, 2023
533e8f5
Merge branch '10054-generate-ea-documents' of github.com:flexion/ef-c…
Sep 22, 2023
1316f25
Merge branch '10054-use-computed-for-consolidated-check' into 10054-g…
Sep 22, 2023
8725ba3
10054: remove references to state.allowExternalConsolidatedGroupFilin…
Sep 22, 2023
6897256
dep-updates: commit to trigger migration
ujahio Sep 22, 2023
3a4a33f
dep-updates: correct type error for notifications state
ujahio Sep 22, 2023
9ae92cd
10054: update test after changing allowExternalConsolidatedGroupFilin…
TomElliottFlexion Sep 22, 2023
e5699d4
Merge branch 'staging' of https://github.com/ustaxcourt/ef-cms into d…
ujahio Sep 25, 2023
a8f9c6e
10054: WIP increase s3 link timeout for generated EAs
TomElliottFlexion Sep 25, 2023
c08d3a7
10054: increase s3 link timeout for generated EAs
TomElliottFlexion Sep 25, 2023
e33523f
Devex: Update NCA integration test to change address on private pract…
ujahio Sep 25, 2023
c1c1e14
design debt: fix mispelling
mmarcotte Sep 25, 2023
3ba4b10
Merge pull request #4002 from flexion/notice-of-change-of-address-fla…
mmarcotte Sep 25, 2023
dd46ef4
Merge branch 'staging' into fix-mispelling
mmarcotte Sep 25, 2023
03162d7
Merge branch 'staging' of https://github.com/ustaxcourt/ef-cms into d…
ujahio Sep 25, 2023
63e2d6f
dep-updates: corrected efcms-docker-image in config.yml
ujahio Sep 25, 2023
aff413f
Merge branch 'staging' of github.com:ustaxcourt/ef-cms into 10054-gen…
codyseibert Sep 25, 2023
b527187
dep-udpates: install @aws-sdk/client-glue @aws-sdk/client-s3
ujahio Sep 25, 2023
e9b3300
Merge pull request #3995 from flexion/dep-updates-08-18-23
jimlerza Sep 26, 2023
c71f2e2
10054: Set isUpload to true so that the user can see correct upload p…
Sep 26, 2023
55376d0
10054: add back mobile view for review access screen
TomElliottFlexion Sep 27, 2023
43ca198
Merge branch 'staging' into 10054-generate-ea-documents
TomElliottFlexion Sep 28, 2023
fa725d6
10054: update test to use jest mock functions
TomElliottFlexion Sep 28, 2023
647e47d
Merge pull request #4018 from flexion/10054-generate-ea-documents
mmarcotte Sep 29, 2023
908e703
Merge branch 'staging' into fix-mispelling
mmarcotte Sep 29, 2023
fd00347
Merge pull request #4003 from ustaxcourt/fix-mispelling
mmarcotte Sep 29, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
version: 2.1
orbs:
git-shallow-clone: guitarrapc/git-shallow-clone@2.6.0
define: &efcms-docker-image $AWS_ACCOUNT_ID.dkr.ecr.us-east-1.amazonaws.com/ef-cms-us-east-1:3.0.14

efcms-docker-image: &efcms-docker-image $AWS_ACCOUNT_ID.dkr.ecr.us-east-1.amazonaws.com/ef-cms-us-east-1:3.0.15

parameters:
run_build_and_deploy:
Expand Down Expand Up @@ -29,15 +30,15 @@ parameters:
type: boolean

referrer:
default: ""
default: ''
type: string

source_bucket:
default: ""
default: ''
type: string

destination_bucket:
default: ""
default: ''
type: string

commands:
Expand Down Expand Up @@ -955,7 +956,7 @@ workflows:
glue-to-test:
when:
or:
- equal: [ glue-to-test-schedule, << pipeline.schedule.name >> ]
- equal: [glue-to-test-schedule, << pipeline.schedule.name >>]
- << pipeline.parameters.run_glue_to_test >>
jobs:
- delete-existing-data:
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ RUN apt-get install -y build-essential
RUN apt-get install -y libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev

ENV JAVA_HOME=/usr/lib/jvm/java-11-openjdk-amd64
RUN curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64-2.13.17.zip" -o "awscliv2.zip" && \
RUN curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64-2.13.20.zip" -o "awscliv2.zip" && \
unzip awscliv2.zip && \
./aws/install && \
rm -rf awscliv2.zip
Expand Down
7 changes: 4 additions & 3 deletions docs/dependency-updates.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,16 +67,17 @@ Below is a list of dependencies that are locked down due to known issues with se

- Keep `@sparticuz/chromium` locked to 112.0.2 and `puppeteer` locked to 19.8.5 as 114+ and 20+ were causing pdf generation timeout bugs. (https://app.zenhub.com/workspaces/flexionef-cms-5bbe4bed4b5806bc2bec65d3/issues/gh/flexion/ef-cms/10087).

- When updating puppeteer or puppeteer core in the project make sure to also match versions in web-api/runtimes/puppeteer/package.json as this is our lambda layer which we use to generate pdfs. Puppeteer and chromium versions should always match between package.json and web-api/runtimes/puppeteer/package.json. Remember to run `npm i` after updating the versions to update the package-lock.json.
- When updating puppeteer or puppeteer core in the project, make sure to also match versions in `web-api/runtimes/puppeteer/package.json` as this is our lambda layer which we use to generate pdfs. Puppeteer and chromium versions should always match between package.json and web-api/runtimes/puppeteer/package.json. Remember to run `npm i` after updating the versions to update the package-lock.json.

#### s3rver
- As of 7/26/2023 there is a high security vulnerability for transitive dependency in s3rver for "fast-xml-parser". This cannot be fixed using the patch method above as it is a dependency of a dependency. Currently waiting for pull request to update fast-xml parser dependency(https://github.com/jamhall/s3rver/pull/813).
- The s3rver package has been abandoned for two years now and is unlikely to be fixed. A community member has forked the repo and fixed the vulnerabilities so we are switching to the forked version here: github:20minutes/s3rver (https://github.com/20minutes/s3rver). This means the s3rver community version is unlikely to be updated however, the original npm package has been abandoned for over 2 years.

### pdfjs-dist

- `pdfjs-dist` has a major version update to ^3.x,x. A devex card has been created to track work being done towards updating. Please add notes and comments to [this card](https://trello.com/c/gjDzhUkb/1111-upgrade-pdfjs-dist).
- `pdfjs-dist` has a major version update to ^3.x,x. A devex card has been created to track work being done towards updating the package. Please add notes and comments to [this card](https://trello.com/c/gjDzhUkb/1111-upgrade-pdfjs-dist).

### Incrementing the Node Cache Key Version

It's rare to need to increment or change the cache key. One reason you may want to do so is if something happens while storing the cache which corrupts it. For example, a few months ago a package failed to install while the cache was being stored. CircleCI had no idea that the installation didn't go according to plan and saved the corrupted cache. In this case, we incremented the cache key version so that CircleCI was forced to reinstall the node dependencies and save them under the new key. The cache key can be updated by searching within config.yml for vX-npm and vX-cypress where X is the current version of the cache key, then increment the version found.
It's rare to need modify cache key. One reason you may want to do so is if a package fails to install properly, and CircleCI, unaware of the failed installation, stores the corrupted cache. In this case, we will need to increment the cache key version so that CircleCI is forced to reinstall the node dependencies and save them using the new key. To update the cache key, locate `vX-npm` and `vX-cypress` (where X represents the current cache key version) in the config.yml file, and then increment the identified version.

2 changes: 1 addition & 1 deletion docs/testing.md
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ The scripts containing the word `readonly` run special smoketests that rely on e

#### Open

The Cypress CLI runner also provides a useful `open` command which will load a browser instance locally which developers can use to watch the tests click through the UI and rewind history.
The Cypress CLI runner also provides a useful `open` command which will load a browser instance locally, which developers can use to watch the tests click through the UI and rewind history.

#### Public

Expand Down
2 changes: 1 addition & 1 deletion iam/terraform/account-specific/main/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,6 @@ terraform {
}

required_providers {
aws = "5.16.1"
aws = "5.17.0"
}
}
2 changes: 1 addition & 1 deletion iam/terraform/environment-specific/main/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,6 @@ terraform {
}

required_providers {
aws = "5.16.1"
aws = "5.17.0"
}
}
Loading