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

Use Node 22, bug fixes #1220

Merged
merged 39 commits into from
Jan 27, 2025
Merged

Use Node 22, bug fixes #1220

merged 39 commits into from
Jan 27, 2025

Conversation

rustyjux
Copy link
Contributor

@rustyjux rustyjux commented Jan 24, 2025

Major change is the use of Node 22.
Also fixes for #1198, #1167

See release notes - bcgov/aps-infra-platform@6fcc678

rustyjux and others added 30 commits October 22, 2024 09:40
* tweak style for gateway list item hover

* ns to gw text
* fix tests to handle redirect away from your products

* upload astra scan results to debug

* fix legal terms URL
* rename namespace-profile endpoint

* update ns --> gw in API descriptions

* delete routes.ts and openapi.yaml (generated during build)
…y logic; improve error handling in namespace service login
upgrade upload-sarif for trivy, try fixing output path for anchore
- Removed redundant log process termination.
- Reordered step to create a GitHub issue if failures are detected.
- Updated the SonarCloud scan action to use the latest SonarQube GitHub action.
Elson9 and others added 7 commits January 6, 2025 13:41
* some console log for testing in dev

* wait for finished fetching

* tweak auth context

* Remove unnecessary namespace.isLoading condition

---------

Co-authored-by: Russell Vinegar <[email protected]>
…ith retry logic; improve error handling in namespace service login"

This reverts commit e964a72.
Copy link
Contributor

🔔 Friendly reminder: The following reviewers still need to review this PR: @Elson9, @ikethecoder

PS: @sdqdadat

@rustyjux rustyjux merged commit 5959497 into test Jan 27, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants