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

Merge into staging #813

Merged
merged 71 commits into from
Feb 6, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
71 commits
Select commit Hold shift + click to select a range
25f0fab
fix: upgrade lru-cache from 10.2.2 to 10.4.3
snyk-bot Aug 21, 2024
bb7a2ef
fix: package.json & package-lock.json to reduce vulnerabilities
snyk-bot Oct 26, 2024
6f6a8b0
build(deps): bump cross-spawn from 7.0.3 to 7.0.6
dependabot[bot] Nov 20, 2024
c5d887d
Merge pull request #754 from DTS-STN/dependabot/npm_and_yarn/cross-sp…
jhslater Nov 26, 2024
596f7e9
fix: package.json & package-lock.json to reduce vulnerabilities
snyk-bot Nov 26, 2024
3783264
Merge branch 'dev' into snyk-upgrade-96756c133acd031a905400101c06390f
jhslater Nov 26, 2024
9d75554
Merge pull request #721 from DTS-STN/snyk-upgrade-96756c133acd031a905…
jhslater Nov 26, 2024
61e5e0f
Merge pull request #746 from DTS-STN/snyk-fix-61c1c903c2cb55fcdba0289…
jhslater Nov 26, 2024
5a0400c
Merge pull request #759 from DTS-STN/snyk-fix-5737eda391cc1ab282e7adf…
jhslater Nov 26, 2024
5570bf0
fix: upgrade @cypress/request from 3.0.1 to 3.0.6
snyk-bot Nov 30, 2024
d515384
fix: upgrade @opentelemetry/exporter-metrics-otlp-proto from 0.52.0 t…
snyk-bot Nov 30, 2024
ba732a3
fix: upgrade jose from 5.9.4 to 5.9.6
snyk-bot Nov 30, 2024
a449bd7
Push error events instead of pageloads on error pages
Charles-Pham Dec 2, 2024
e83e1f5
Merge pull request #766 from DTS-STN/aa-error-page
Charles-Pham Dec 3, 2024
f04932b
Fix beforeInteractive lint warning
Charles-Pham Dec 3, 2024
4586a6c
Merge pull request #778 from DTS-STN/aa-error-page
Charles-Pham Dec 5, 2024
9cafafa
Change profile to typescript
Charles-Pham Dec 12, 2024
921c86c
Temporarily handle some variables being potentially missing
Charles-Pham Dec 12, 2024
c86b268
Yet more fixes.
Charles-Pham Dec 12, 2024
f8c73fd
Remove potential to be an internal react object
Charles-Pham Dec 12, 2024
35e49c4
Change return type to ReactNode
Charles-Pham Dec 13, 2024
1f0165c
Merge pull request #779 from DTS-STN/typescript-conversion
Charles-Pham Dec 13, 2024
accc921
build(deps): bump nanoid from 3.3.7 to 3.3.8
dependabot[bot] Dec 14, 2024
af36e9f
Convert security settings and decision review to typescript
Charles-Pham Dec 16, 2024
9010680
Remove casts
Charles-Pham Dec 17, 2024
fa5b645
Convert acronym to typescript and totally not a way to rerun tests.
Charles-Pham Dec 17, 2024
19b0e58
Merge pull request #781 from DTS-STN/typescript-conversion
Charles-Pham Dec 17, 2024
38097b1
Covert library functions, document, and dashboard to typescript
Charles-Pham Dec 17, 2024
b416129
fix types
Charles-Pham Dec 17, 2024
dffa2b1
Undo unintended change
Charles-Pham Dec 18, 2024
7ed55bf
Merge pull request #782 from DTS-STN/typescript-conversion
Charles-Pham Dec 19, 2024
a2fac8e
Allow AEM folder to be specified
Charles-Pham Dec 19, 2024
eeec649
update example config
Charles-Pham Dec 19, 2024
a2dcdd7
Totally a functional change and not a way to get tests to try to reru…
Charles-Pham Dec 19, 2024
aa9fc88
update helmfile
Charles-Pham Dec 19, 2024
bba713f
update config
Charles-Pham Dec 20, 2024
e515e95
Merge pull request #783 from DTS-STN/typescript-conversion
Charles-Pham Dec 20, 2024
b2ae672
Convert layout to typescript
Charles-Pham Dec 31, 2024
a05ad69
Merge pull request #784 from DTS-STN/typescript-conversion
Charles-Pham Jan 3, 2025
a565c21
Change _app to typescript
Charles-Pham Jan 6, 2025
50eea36
fix: package.json & package-lock.json to reduce vulnerabilities
snyk-bot Jan 7, 2025
663f283
Convert metrics and refresh-msca to typescript
Charles-Pham Jan 8, 2025
c56ad0b
Merge pull request #787 from DTS-STN/typescript-conversion
Charles-Pham Jan 10, 2025
25b657e
Fix bad AA tags and try fixing error page event
Charles-Pham Jan 13, 2025
4fef30d
Merge pull request #761 from DTS-STN/snyk-upgrade-d82022707fbba16ed8d…
jhslater Jan 20, 2025
73a7c5e
Merge pull request #762 from DTS-STN/snyk-upgrade-acadd823b29181a4cff…
jhslater Jan 20, 2025
48b4a7d
Merge pull request #765 from DTS-STN/snyk-upgrade-72585213c99191f3859…
jhslater Jan 20, 2025
41b1a27
Merge pull request #786 from DTS-STN/snyk-fix-a6ca34352a36e43388e24b4…
jhslater Jan 20, 2025
c5a0b7e
Merge pull request #780 from DTS-STN/dependabot/npm_and_yarn/nanoid-3…
jhslater Jan 20, 2025
235c6b6
upgrade @opentelemetry/exporter-jaeger from 1.25.1 to 1.27.0 (manual …
jhslater Jan 20, 2025
907f0db
Merge pull request #788 from DTS-STN/exporter-jaeger_1-27-0
jhslater Jan 20, 2025
9b7547c
upgrade @opentelemtry/sdk-node from 0.52.0 to 0.54.2 (manually becaus…
jhslater Jan 20, 2025
1173981
Merge pull request #789 from DTS-STN/upgrade_sdk-node-0-54-2
jhslater Jan 20, 2025
4c4caea
fix: upgrade zod from 3.23.8 to 3.24.1 (#790)
shewood Jan 21, 2025
21c1488
fix: upgrade markdown-to-jsx from 7.4.7 to 7.7.2 (#791)
shewood Jan 22, 2025
697efcd
Fix aaTitle and missed dupe
Charles-Pham Jan 22, 2025
49d62f2
Upgrade @fortawesome 6.5.2 to 6.7.2 (manually because the Snyk PR was…
abalogh-esdc Jan 23, 2025
2944d4d
Merge pull request #797 from DTS-STN/aa-fix
Charles-Pham Jan 28, 2025
af0ba42
fix store results
Charles-Pham Feb 3, 2025
189d2c4
increase timeout to 60 seconds
Charles-Pham Feb 3, 2025
068128d
try adding some retries
Charles-Pham Feb 3, 2025
0c0c757
Merge pull request #805 from DTS-STN/test/build-test
Charles-Pham Feb 4, 2025
966679e
add ui_locales query parameter
Charles-Pham Feb 4, 2025
c421dc5
Merge pull request #804 from DTS-STN/logout-locale
Charles-Pham Feb 5, 2025
507a1dc
use correct timeout config
Charles-Pham Feb 5, 2025
3062cba
try to retry on status code failures to handle intermittent problems
Charles-Pham Feb 5, 2025
bd337de
more narrowly focus retrying and timeouts to the initial visit
Charles-Pham Feb 5, 2025
bff21e1
increase timeout back up
Charles-Pham Feb 5, 2025
8aa44e2
add back retries
Charles-Pham Feb 5, 2025
960cf47
Merge pull request #807 from DTS-STN/fix/test-timeout
Charles-Pham Feb 6, 2025
642fc11
Merge branch 'staging' into dev
Charles-Pham Feb 6, 2025
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
2 changes: 2 additions & 0 deletions .env.example
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
AEM_GRAPHQL_ENDPOINT = "test.com"
# Code suggests this is optional but it is actually not because of changes in the structure of the response
AEM_GRAPHQL_FOLDER = "test_folder/path"
LOGGING_LEVEL = " info or debug"
MSCA_BASE_URL = "MSCA base url"
MSCA_EQ_BASE_URL = "MSCA base url with eq extension"
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/dast.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,16 +19,18 @@ jobs:
CI: true

- name: Docker Build
run: docker build --build-arg AEM_GRAPHQL_ENDPOINT=${{secrets.AEM_GRAPHQL_ENDPOINT}} -t secure-client-hub .
run: docker build --build-arg AEM_GRAPHQL_ENDPOINT=${{secrets.AEM_GRAPHQL_ENDPOINT}} AEM_GRAPHQL_FOLDER=${{secrets.AEM_GRAPHQL_FOLDER}} -t secure-client-hub .
env:
CI: true
AEM_GRAPHQL_ENDPOINT: ${{secrets.AEM_GRAPHQL_ENDPOINT}}
AEM_GRAPHQL_FOLDER: ${{secrets.AEM_GRAPHQL_FOLDER}}

- name: Docker run
run: docker run -d -p 3000:3000 secure-client-hub
env:
CI: true
AEM_GRAPHQL_ENDPOINT: ${{secrets.AEM_GRAPHQL_ENDPOINT}}
AEM_GRAPHQL_FOLDER: ${{secrets.AEM_GRAPHQL_FOLDER}}

- name: OWASP ZAP FULL Scan
uses: zaproxy/[email protected]
Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/default-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
AUTH_DISABLED: true

- name: Store Results
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: unit-test-results
path: coverage
Expand All @@ -78,6 +78,7 @@ jobs:
env:
CI: true
AEM_GRAPHQL_ENDPOINT: ${{secrets.AEM_GRAPHQL_ENDPOINT}}
AEM_GRAPHQL_FOLDER: ${{secrets.AEM_GRAPHQL_FOLDER}}
AUTH_DISABLED: true

- name: Cypress end-to-end 🧪
Expand All @@ -86,6 +87,7 @@ jobs:
CI: true
NODE_ENV: production
AEM_GRAPHQL_ENDPOINT: ${{secrets.AEM_GRAPHQL_ENDPOINT}}
AEM_GRAPHQL_FOLDER: ${{secrets.AEM_GRAPHQL_FOLDER}}
NEXTAUTH_SECRET: ${{secrets.NEXTAUTH_SECRET}}
NEXTAUTH_URL: ${{secrets.NEXTAUTH_URL}}
CLIENT_SECRET: ${{secrets.CLIENT_SECRET}}
Expand Down
4 changes: 4 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ ARG LOGGING_LEVEL=info
ENV LOGGING_LEVEL=$LOGGING_LEVEL
ARG AEM_GRAPHQL_ENDPOINT=https://www.canada.ca/graphql/execute.json/decd-endc/
ENV AEM_GRAPHQL_ENDPOINT=$AEM_GRAPHQL_ENDPOINT
ARG AEM_GRAPHQL_FOLDER
ENV AEM_GRAPHQL_FOLDER=$AEM_GRAPHQL_FOLDER
ARG AUTH_ECAS_BASE_URL
ENV AUTH_ECAS_BASE_URL=$AUTH_ECAS_BASE_URL
ARG MSCA_BASE_URL
Expand Down Expand Up @@ -99,6 +101,8 @@ ARG LOGGING_LEVEL=info
ENV LOGGING_LEVEL=$LOGGING_LEVEL
ARG AEM_GRAPHQL_ENDPOINT
ENV AEM_GRAPHQL_ENDPOINT=$AEM_GRAPHQL_ENDPOINT
ARG AEM_GRAPHQL_FOLDER
ENV AEM_GRAPHQL_FOLDER=$AEM_GRAPHQL_FOLDER
ARG MSCA_BASE_URL
ENV MSCA_BASE_URL=$MSCA_BASE_URL
ARG MSCA_EQ_BASE_URL
Expand Down
6 changes: 2 additions & 4 deletions components/BenefitTasks.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -11,12 +11,10 @@ interface Tasks {
id: string
}

interface TaskListProps {
export interface TaskListProps {
title: string
dataCy: string
tasks: Tasks[]
}

interface BenefitTasksProps {
locale: string
taskList: TaskListProps
Expand Down Expand Up @@ -81,7 +79,7 @@ const BenefitTasks = ({
? 'noopener noreferrer'
: undefined
}
data-gc-analytics-customclick={`${refPageAA} ${acronym} ${taskList.title}:${task.id}`}
data-gc-analytics-customclick={`${refPageAA} ${acronym}:${task.id}`}
className="flex items-center rounded-sm py-1 text-deep-blue-dark underline hover:text-blue-hover focus:outline-1 focus:outline-blue-hover"
>
<span className="static text-xl font-normal">
Expand Down
4 changes: 2 additions & 2 deletions components/Button.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ interface ButtonProps {
iconEnd?: boolean
href?: string
type?: 'submit' | 'reset' | 'button'
onClick: MouseEventHandler<HTMLElement>
onClick?: MouseEventHandler<HTMLElement>
disabled?: boolean
className?: string
attributes?: { [key: string]: string }
Expand All @@ -27,7 +27,7 @@ const Button = ({
iconEnd,
href = 'no ref',
type,
onClick,
onClick = () => {},
disabled,
className,
attributes,
Expand Down
4 changes: 0 additions & 4 deletions components/Card.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ interface AlertProps {
type: string
alertHeading: string
alertBody: string
alert_icon_alt_text: string
alert_icon_id: string
}
interface CardProps {
cardTitle: string
Expand All @@ -31,8 +29,6 @@ const Card = ({
type: '',
alertHeading: '',
alertBody: '',
alert_icon_alt_text: '',
alert_icon_id: '',
},
],
locale,
Expand Down
8 changes: 4 additions & 4 deletions components/Footer.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,24 +10,24 @@ interface Links {
id: string
text: string
href: string
onClick: () => void
onClick?: () => void
}

interface FooterProps {
id: string
lang: 'en' | 'fr' | 'und'
btnLink: string
error: boolean
error?: boolean
contactLink: string
brandLinks: Links[]
target: string
target?: string
}

const Footer = ({
id = 'mscaPlaceholder',
lang = 'en',
btnLink,
error,
error = false,
contactLink = 'https://www.canada.ca/en/contact.html',
brandLinks = [
{
Expand Down
8 changes: 4 additions & 4 deletions components/Header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ interface SearchProps {
}

interface MenuProps {
onSignOut: () => void
onSignOut?: () => void
menuList: MenuList[]
}

Expand All @@ -21,8 +21,8 @@ interface MenuList {
key: string
value: string
path: string
showIcon: boolean
onSignOut: () => void
showIcon?: boolean
onSignOut?: () => void
}

interface TopNavProps {
Expand All @@ -32,7 +32,7 @@ interface TopNavProps {
displayAlternateLink: boolean
}

interface BreadcrumbItemProps {
export interface BreadcrumbItemProps {
text: string
link: string
}
Expand Down
Loading