Skip to content

PB-1877 Add test link to PR #patch - #40

Merged
asteiner-swisstopo merged 1 commit into
masterfrom
feat-PB-1877-ci-cd-test-link-in-pr
Sep 22, 2025
Merged

asteiner-swisstopo merged 1 commit into
masterfrom
feat-PB-1877-ci-cd-test-link-in-pr

Conversation

@asteiner-swisstopo

@asteiner-swisstopo asteiner-swisstopo commented Sep 19, 2025

Copy link
Copy Markdown
Contributor

This is adapted from the web-mapviewer.

We only have feature branches and a master branch. So no conditionals needed.

The preview link points to something like https://sys-docs.dev.bgdi.ch/preview/feat-PB-1877-my-title/index.html which points to S3 bucket s3://tech-docs-dev-swisstopo.

The corresponding CodeBuild project, S3 bucket and CloudFront configs are set up in https://github.com/geoadmin/infra-terraform-bgdi/pull/1179.

Test link

This is adapted from the web-mapviewer.

We only have feature branches and a master branch. So no conditionals needed.

The preview link points to something like https://sys-docs.dev.bgdi.ch/preview/feat-PB-1877-my-title/index.html which points to S3 bucket `s3://tech-docs-dev-swisstopo`.

The corresponding CodeBuild project, S3 bucket and CloudFront configs are set up in geoadmin/infra-terraform-bgdi#1179.

@ltflb-bgdi ltflb-bgdi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@asteiner-swisstopo asteiner-swisstopo changed the title PB-1877 Add test link to PR PB-1877 Add test link to PR #patch Sep 22, 2025
@asteiner-swisstopo
asteiner-swisstopo merged commit fbc6116 into master Sep 22, 2025
2 checks passed
@asteiner-swisstopo
asteiner-swisstopo deleted the feat-PB-1877-ci-cd-test-link-in-pr branch September 22, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants