Skip to content

Feature nuxt v3 upgrade#285

Open
seeker25 wants to merge 252 commits intomainfrom
feature-nuxt-v3-upgrade
Open

Feature nuxt v3 upgrade#285
seeker25 wants to merge 252 commits intomainfrom
feature-nuxt-v3-upgrade

Conversation

@seeker25
Copy link
Copy Markdown
Collaborator

@seeker25 seeker25 commented Nov 28, 2025

No description provided.

deetz99 and others added 30 commits September 19, 2025 12:11
@deetz99
add eslint
0ca24b2
@deetz99
add layers, i18n, update env
0533885
@deetz99
setup unit/e2e tests
3de635d
@deetz99
update vault env
c33d9c4
@deetz99
add e2e to ci
801f0eb
@deetz99
add modal example
dc2fe80
@deetz99
set package version to 2.0.0-beta
9ec7e09
@deetz99
add preinstall script for pnpm only
fea67f8
@deetz99
fix app name in nuxt config
94aeef1
@deetz99
remove fas url from config, baseUrl is same
8cacc4e
@deetz99
add component folder
392db05
@deetz99
Merge pull request #270 from deetz99/24412-nuxt-project-setup
612e6d2 
24412 nuxt project setup
@seeker25
Update README.md
b85edf8
@seeker25
Update README.md
adaba17
@seeker25
Update README.md
081d1a8
@seeker25
Update README.md
36156a9
@seeker25
Update README.md
75fb999
@thorwolpert
fix: add missing .npmrc to set hoisting. Update package and lockfile (#…
88e8be5 
…272)
@deetz99
add DateRangeFilter component
0200405
@deetz99
pr review fixes
1e082b6
@deetz99
update comment
c80fb04
@deetz99
Merge pull request #273 from deetz99/30793-daterangefilter
4d94006 
30793 - add DateRangeFilter component
@deetz99
datepicker component & tests
660d4df
@deetz99
icontooltip component
077e0a2
@deetz99
remove placeholder component
c41efaa
@deetz99
Merge pull request #274 from deetz99/30793-datepicker-icontooltip
cda0521 
30793 datepicker icontooltip
@deetz99
StatusMenu and StatusList components
8f3c5ba
@deetz99
add comment
22a3dca
@deetz99
Merge pull request #275 from deetz99/30793-statusmenu-statuslist
c976d48 
30793 - StatusMenu and StatusList components
@deetz99
column filter component (#276)
3098321
@bcregistry-sre
Copy link
Copy Markdown
Collaborator

bcregistry-sre commented Mar 3, 2026

Temporary Url for review: https://pay-web-dev--pr-285-xi553ou5.web.app

@ochiu
Copy link
Copy Markdown
Collaborator

ochiu commented Mar 3, 2026

/gcbrun

@bcregistry-sre
Copy link
Copy Markdown
Collaborator

bcregistry-sre commented Mar 3, 2026

Temporary Url for review: https://pay-web-dev--pr-285-xi553ou5.web.app

@anisshhbatrra
fixed test automation (#312)
6bff652 
Co-authored-by: Your Name <your.email@example.com>
github-advanced-security[bot]
github-advanced-security bot found potential problems Mar 6, 2026
View reviewed changes
test-automation/globalSetup.js Fixed

async function performLoginAndSaveSession(page, context, baseURL, loginType, username, password) {
await page.goto(baseURL)
console.log(`Global setup: loginType=${loginType}, username=${username}`)

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information High test

This logs sensitive data returned by process environment as clear text.
This logs sensitive data returned by process environment as clear text.
This logs sensitive data returned by process environment as clear text.
This logs sensitive data returned by process environment as clear text.

Copilot Autofix

AI 29 days ago

In general, to fix clear-text logging of sensitive information, remove sensitive values from log messages or replace them with non-sensitive, redacted, or generalized information. Keep logging high-level status (e.g., loginType, environment, success/failure) without printing secrets or identifiers derived from environment variables used for authentication.

For this specific issue, the problematic sink is on line 106:

console.log(`Global setup: loginType=${loginType}, username=${username}`)

The best fix that preserves functionality is to stop logging the raw username. We can still log loginType and, if helpful, a non-sensitive indicator that a username was used or a redacted form (e.g., masking all but the first character). Since we must avoid assuming extra utility functions elsewhere, the simplest and safest change is to remove username from the log entirely:

console.log(`Global setup: Starting login using loginType=${loginType}`)

This ensures no environment-derived credential is written to logs while maintaining useful debug information. No new imports or helper methods are required; the change is limited to the single log line in performLoginAndSaveSession in test-automation/globalSetup.js.

Suggested changeset 1
test-automation/globalSetup.js

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/test-automation/globalSetup.js b/test-automation/globalSetup.js
--- a/test-automation/globalSetup.js
+++ b/test-automation/globalSetup.js
@@ -103,7 +103,7 @@
 
 async function performLoginAndSaveSession(page, context, baseURL, loginType, username, password) {
   await page.goto(baseURL)
-  console.log(`Global setup: loginType=${loginType}, username=${username}`)
+  console.log(`Global setup: Starting login using loginType=${loginType}`)
 
 
   const loginPage = new LoginPage(page)
EOF
@@ -103,7 +103,7 @@

async function performLoginAndSaveSession(page, context, baseURL, loginType, username, password) {
await page.goto(baseURL)
console.log(`Global setup: loginType=${loginType}, username=${username}`)
console.log(`Global setup: Starting login using loginType=${loginType}`)


const loginPage = new LoginPage(page)
Copilot is powered by AI and may make mistakes. Always verify output.
@seeker25 seeker25 committed this autofix suggestion 29 days ago.
Copy link
Copy Markdown
Collaborator Author

@seeker25 seeker25 Mar 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@seeker25 @github-advanced-security
Potential fix for code scanning alert no. 12: Clear-text logging of s…
7183bfe 
…ensitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@bcregistry-sre

This comment was marked as outdated.

Sign in to view
ochiu added 2 commits March 12, 2026 15:50
@ochiu
32196, 32534 - more UXA (#316)
03bbfe3
@ochiu
32148 - QA fixes (#317)
5672d8e 
* fix decimal point validation

* button hide / show logic update and linting

* lint

* lint, test fixes / updates
@ochiu
Copy link
Copy Markdown
Collaborator

ochiu commented Mar 17, 2026

/gcbrun

@bcgov bcgov deleted a comment from bcregistry-sre Mar 17, 2026
@bcgov bcgov deleted a comment from bcregistry-sre Mar 17, 2026
@bcregistry-sre
Copy link
Copy Markdown
Collaborator

bcregistry-sre commented Mar 17, 2026

Temporary Url for review: https://pay-web-dev--pr-285-4pbohven.web.app

@Jxio Jxio mentioned this pull request Mar 24, 2026
Merged
@ochiu
32196 - EFT UX / QA clean up (#321)
72fe0f4 
* 32196 - EFT UX / QA clean up

* test fix
@ochiu
Copy link
Copy Markdown
Collaborator

ochiu commented Mar 24, 2026

/gcbrun

@bcregistry-sre
Copy link
Copy Markdown
Collaborator

bcregistry-sre commented Mar 24, 2026

Temporary Url for review: https://pay-web-dev--pr-285-4pbohven.web.app

@ochiu
32196 - EFT QA UXA 2 (#322)
2d55dc3 
* 32196 - EFT UX / QA clean up

* test fix

* 32196 - EFT UXA Review 2
@ochiu
Copy link
Copy Markdown
Collaborator

ochiu commented Mar 25, 2026

/gcbrun

@bcregistry-sre
Copy link
Copy Markdown
Collaborator

bcregistry-sre commented Mar 25, 2026

Temporary Url for review: https://pay-web-dev--pr-285-4pbohven.web.app

anisshhbatrra and others added 3 commits March 30, 2026 09:13
@ochiu
Copy link
Copy Markdown
Collaborator

ochiu commented Mar 31, 2026

/gcbrun

@bcregistry-sre
Copy link
Copy Markdown
Collaborator

bcregistry-sre commented Mar 31, 2026

Temporary Url for review: https://pay-web-dev--pr-285-4pbohven.web.app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Jxio Jxio Awaiting requested review from Jxio Jxio is a code owner

@ochiu ochiu Awaiting requested review from ochiu ochiu is a code owner

1 more reviewer

@anisshhbatrra anisshhbatrra anisshhbatrra approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Do Not Merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@seeker25 @bcregistry-sre @ochiu @github-advanced-security @anisshhbatrra @deetz99 @thorwolpert @circuit-breaker-bc @Jxio