Skip to content

chore(ci): drop vanity url from prod workflow#838

Merged
DerekRoberts merged 1 commit intomainfrom
chore/ci/routeSplit
Jul 29, 2025
Merged

chore(ci): drop vanity url from prod workflow#838
DerekRoberts merged 1 commit intomainfrom
chore/ci/routeSplit

Conversation

@DerekRoberts
Copy link
Member

@DerekRoberts DerekRoberts commented Jul 29, 2025

Remove PROD URL from workflows. This is now happening independently. The regular workflows will deploy non-vanity URLs, which can be ignored.


Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Copilot AI review requested due to automatic review settings July 29, 2025 18:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the vanity URL configuration from the production deployment workflow as it is now being handled independently. The change simplifies the workflow by removing the hardcoded production URL parameter.

  • Remove the url: fom.nrs.gov.bc.ca parameter from the production workflow
  • Allows workflows to deploy with non-vanity URLs that can be safely ignored

@DerekRoberts DerekRoberts self-assigned this Jul 29, 2025
@DerekRoberts DerekRoberts moved this from New to Active in DevOps (NR) Jul 29, 2025
@github-project-automation github-project-automation bot moved this from Active to Waiting in DevOps (NR) Jul 29, 2025
@DerekRoberts DerekRoberts merged commit ddf99b9 into main Jul 29, 2025
23 checks passed
@DerekRoberts DerekRoberts deleted the chore/ci/routeSplit branch July 29, 2025 18:43
@github-project-automation github-project-automation bot moved this from Waiting to Done in DevOps (NR) Jul 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants