generated from NHSDigital/nhs-notify-repository-template
-
Notifications
You must be signed in to change notification settings - Fork 2
Feature/ccm 13610 letter request event performance second branch #328
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
Closed
Vlasis-Perdikidis
wants to merge
155
commits into
main
from
feature/CCM-13610-letterRequestEventPerformance_secondBranch
Closed
Changes from 145 commits
Commits
Show all changes
155 commits
Select commit
Hold shift + click to select a range
e5cfae1
script
Namitha-Prabhu 0d7458d
CCM-12180: acceptance stage internal invoke
masl2 4999505
CCM-12180: correct infraRepo name param
masl2 c021822
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
masl2 104851e
CCM-12180: add override project and role
masl2 d6c1239
CCM-12180: override project
masl2 a424ceb
CCM-12180: remove callerId param
masl2 29fdc70
fix tests
Namitha-Prabhu 038076e
remove generate-output
Namitha-Prabhu 223d427
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
Namitha-Prabhu d18d495
tests
Namitha-Prabhu d161b85
merge
Namitha-Prabhu 787face
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
Namitha-Prabhu b154794
merge
Namitha-Prabhu 1d53900
Add upsert lambda
francisco-videira-nhs 4334697
Fix names
francisco-videira-nhs 132326d
Fix unit test
francisco-videira-nhs 036c319
Fix unit test dates
francisco-videira-nhs d6602c3
Fix lambda handler name export
francisco-videira-nhs 10326e9
merge
Namitha-Prabhu d78a8c7
merge
Namitha-Prabhu 92d4d26
Add event-schemas-letter-rendering
francisco-videira-nhs b040535
Add GitHub NPM registry configuration to .npmrc
m-houston ffb8d5c
Add version check and Node.js setup to CI workflow
m-houston c307fc9
TO REVERT AFTER TEST: conditional check for event schema version update
m-houston 46c8402
Refactor event schema version checks in CI workflow
m-houston f1444fd
Add GitHub NPM registry URL to Node.js setup in workflows
m-houston 59d8846
Log npm response
m-houston d7a665c
Add NODE_AUTH_TOKEN to GitHub Actions environment for NPM access
m-houston 5741a86
Revert "Add GitHub NPM registry URL to Node.js setup in workflows"
m-houston c427d03
Move NODE_AUTH_TOKEN to npm view step in CI workflow
m-houston 8981a44
Reapply "Add GitHub NPM registry URL to Node.js setup in workflows"
m-houston fbf2e9d
Add NODE_AUTH_TOKEN to NPM install steps in workflows
m-houston 113bbe5
Re-enable conditional check on published package check
m-houston 032e48f
Add NODE_AUTH_TOKEN input for GitHub package registry access in workf…
m-houston f24f37a
Add packages:read to permissions
m-houston 333f352
Merge branch 'main' into feature/fix-github-npm-registry-access
m-houston 0d41d64
Merge remote-tracking branch 'origin/main' into feature/CCM-12997
francisco-videira-nhs 6400400
delete old test
francisco-videira-nhs dcead20
tests
Namitha-Prabhu 6a7efa3
Merge remote-tracking branch 'origin/feature/fix-github-npm-registry-…
francisco-videira-nhs 116c323
fix unit tests
Namitha-Prabhu 1d35f02
fix unit tests
Namitha-Prabhu 4001e32
fix trivy scan step
francisco-videira-nhs ffdf716
fix unit tests
Namitha-Prabhu 7e8533f
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
masl2 c5d9b9e
dependencies
masl2 20c5fd0
fix jest configurations
masl2 66eb971
Fix gh pkg registry test stage
francisco-videira-nhs b8b45ac
Merge remote-tracking branch 'origin/main' into feature/CCM-12997
francisco-videira-nhs 1fbbb52
event package version
masl2 64d0fe2
fix local npmrc
francisco-videira-nhs 7c57d35
lint fixes
Namitha-Prabhu fb0ebd9
lint fixes
Namitha-Prabhu 9aaab4c
Merge branch 'feature/CCM-12180-TestsOnPipeline' of https://github.co…
Namitha-Prabhu 52bf92b
remove duplicate variable defs
masl2 c8c3fd8
Merge branch 'feature/CCM-12180-TestsOnPipeline' of https://github.co…
Namitha-Prabhu 4750470
remove test archive
Namitha-Prabhu d3aba34
suppress test data false positive
masl2 1adba99
imports
Namitha-Prabhu df1261c
checks for supplier data
Namitha-Prabhu 7f06b68
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
Namitha-Prabhu 49fd69b
tests
Namitha-Prabhu d0c63e1
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
masl2 5685221
merge
Namitha-Prabhu 7f920cc
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
Namitha-Prabhu 7636dda
Merge branch 'feature/CCM-12180-TestsOnPipeline' of https://github.co…
Namitha-Prabhu 3e38147
version
Namitha-Prabhu 0843041
lock file
Namitha-Prabhu 93f72a7
lock file
Namitha-Prabhu f531aaf
fix
Namitha-Prabhu ecb88e8
fix
Namitha-Prabhu 6bed21e
fix
Namitha-Prabhu 5f30fa4
update test to reflect version
masl2 85233bf
package lock
masl2 ca58f3b
Fix unit test timestamps, add source
francisco-videira-nhs 7b36de2
replace js-yaml with yaml
masl2 48b242b
downgrade yargs due to y18n dep
masl2 4513c92
missing deps
masl2 65481fd
Merge remote-tracking branch 'origin/main' into feature/CCM-12997
francisco-videira-nhs 8a74c03
Add notification parsing logic
francisco-videira-nhs cb29729
fix tf var
francisco-videira-nhs 2715285
Fix ddb permissions
francisco-videira-nhs 5e1c732
update versions
Namitha-Prabhu 3b5a35b
update versions
Namitha-Prabhu b1a5d66
fetch env variable
Namitha-Prabhu 415d07a
fetch env variable
Namitha-Prabhu d9a5a14
PR number
Namitha-Prabhu a5ef09e
PR number
Namitha-Prabhu d6e80aa
Add latest version event schema letter rendering
francisco-videira-nhs 595d4b6
Add trying v1 parse if v2 fails
francisco-videira-nhs d470154
add flag for execute endpoint
masl2 78ece30
Add subject
francisco-videira-nhs 52513c2
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
Namitha-Prabhu a8fa07e
Split upsert operations
francisco-videira-nhs 6040c12
Merge remote-tracking branch 'origin/main' into feature/CCM-12997
francisco-videira-nhs b1e1195
Letter variant map to vars
francisco-videira-nhs fdd73fc
Revert supplierStatusSk
francisco-videira-nhs 05a6b2b
Merge remote-tracking branch 'origin/main' into feature/CCM-12997
francisco-videira-nhs 10ec08f
Fix dependencies
francisco-videira-nhs 9158018
merge
Namitha-Prabhu b7b91d2
Merge branch 'main' into feature/CCM-12180-TestsOnPipeline
Namitha-Prabhu b9d364e
trivy job setup of node for nhs packages
masl2 52b3af0
lock
masl2 f62a004
lock
masl2 7196cba
verbose log level
masl2 832f20e
try node setup and token env
masl2 1947273
node setup
masl2 6f6cd33
acceptance deps
masl2 4e060b6
acceptance deps
masl2 7efebb6
another go
masl2 c232df7
another go
masl2 547f6b1
Merge branch 'main' into feature/CCM-12997
masl2 e42d26c
CCM-13697: Letters Key and Source Data Item (#323)
masl2 448702b
missing close brace
masl2 c68acc2
merge feature/CCM-12997 into my branch
Vlasis-Perdikidis 6e13097
minor refactor upsert unit tests
francisco-videira-nhs 1eecf9b
remove dup line
francisco-videira-nhs 6e86b9d
Merge remote-tracking branch 'origin/main' into feature/CCM-12180-Tes…
francisco-videira-nhs 4693211
fix workspace for supplier test data
francisco-videira-nhs 25d2d8f
merge latest from feature/CCM-12180-TestsOnPipeline into my branch
Vlasis-Perdikidis 018febb
merge latest from feature/CCM-12997
Vlasis-Perdikidis ccea30d
Merge branch 'main' into feature/CCM-13610-letterRequestEventPerforma…
Vlasis-Perdikidis d8db90f
add logs to upsert lambda
Vlasis-Perdikidis 82f6be3
trigger another build
Vlasis-Perdikidis 6a5bd82
add fingerprints to gitleaksignore
Vlasis-Perdikidis f6c2d06
add more logging in upsert lambda
Vlasis-Perdikidis 2162ff5
merge with main
Vlasis-Perdikidis 777eb40
working prepared letter performance test
Vlasis-Perdikidis 0d298ee
correct test description. Remove unnecessary changes. Remove unused i…
Vlasis-Perdikidis 38a655b
Merge remote-tracking branch 'origin/main' into feature/CCM-12997
francisco-videira-nhs ce95fab
gitleaksignore
stevebux ab8ec27
ignore gitLeak
Vlasis-Perdikidis a6e99c2
bump package version
francisco-videira-nhs 63bd2f7
Merge remote-tracking branch 'refs/remotes/origin/main' into feature/…
francisco-videira-nhs 5d80dd3
resolve npm audit vulnerabilities
Vlasis-Perdikidis a76a0fe
increase version of internal/events/package.json
Vlasis-Perdikidis f935c9b
merge feature/CCM-12997 into my branch
Vlasis-Perdikidis 2122890
add npm.pkg.github.com in .npmrc
Vlasis-Perdikidis 59acd7c
populate NODE_AUTH_TOKEN for unit-test
Vlasis-Perdikidis 3e468c0
rebase with main again
Vlasis-Perdikidis 9b09f57
merge with main to get lint changes
Vlasis-Perdikidis 6893aef
remove files irrelevant to my PR
Vlasis-Perdikidis 06b42e5
fix linting errors
Vlasis-Perdikidis a880056
set NODE_AUTH_TOKEN in test-unit stage
Vlasis-Perdikidis 55e3bd8
set NODE_AUTH_TOKEN in linting and typecheck stages
Vlasis-Perdikidis a7ed892
revert incorrect changes
Vlasis-Perdikidis d804997
remove entry from .npmrc
Vlasis-Perdikidis ab6ded4
add clean scripts in root package.json
Vlasis-Perdikidis 289c3c2
fix failing upsert-handler test
Vlasis-Perdikidis a75180b
Merge branch 'main' into feature/CCM-13610-letterRequestEventPerforma…
Vlasis-Perdikidis 70a0d02
address PR comments
Vlasis-Perdikidis b2e7c6f
add back gitLeakIgnore line
Vlasis-Perdikidis a6ad5c2
fix linting errors
Vlasis-Perdikidis 9afd9f1
rebase with main
Vlasis-Perdikidis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,3 +3,4 @@ API_KEY= | |
| HEADERAUTH= | ||
| PR_NUMBER=prxx # remove if needs to run against main | ||
| NHSD_APIM_TOKEN= | ||
| GITHUB_TOKEN= | ||
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,2 +1,3 @@ | ||
| # Package is scoped under @org, set registry for that scope | ||
| @nhsdigital:registry=https://npm.pkg.github.com | ||
| @org:registry=https://npm.pkg.github.com | ||
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.