Skip to content

DAGE-1226: Move away from Netlify - #1241

Merged
toblea merged 2 commits into
masterfrom
DAGE-1226_NewHosting
Aug 18, 2026
Merged

DAGE-1226: Move away from Netlify#1241
toblea merged 2 commits into
masterfrom
DAGE-1226_NewHosting

Conversation

@toblea

@toblea toblea commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Description of change

The hosting costs at Netlify have been bumped and are now extremely high. We need to find a new hosting model for it. This Pull Request adds two new GitHub Actions – one that builds to a new "public" website and one that builds to a "preview" website.

https://qlik-dev.atlassian.net/browse/DAGE-1226

Manual QA steps

The files will probably need to be merged to main before the actions are available to execute. Though, they are both only manually started.

Risks

Low risk to start with, since the actions are manually executed.

Rollback steps

Revert this branch, or just remove the YAML files.

@netlify

netlify Bot commented Aug 18, 2026

Copy link
Copy Markdown

Deploy Preview for stitchdocs ready!

Name Link
🔨 Latest commit b368fe1
🔍 Latest deploy log https://app.netlify.com/projects/stitchdocs/deploys/6a84400862730f00081f49b9
😎 Deploy Preview https://deploy-preview-1241--stitchdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI 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.

Pull request overview

This PR introduces a new GitHub Actions-based deployment path to replace Netlify hosting by building the Jekyll site in CI and deploying either a PR-scoped preview to an S3 prefix (fronted by CloudFront) or a main-site build to the production S3 bucket.

Changes:

  • Add a “preview” workflow intended to build a PR preview and publish it to s3://qlikhelp-stitch-preview/pr-<PR_NUMBER> with a PR comment containing the preview link.
  • Add a “main” workflow to build the site and sync the build output to the production S3 bucket (plus upload the build as an artifact).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 8 comments.

File Description
.github/workflows/build-preview.yml Adds a manually-triggered PR preview deploy (and intended cleanup) workflow targeting S3/CloudFront.
.github/workflows/build-main.yml Adds a manually-triggered production build-and-sync workflow targeting the main S3 bucket.
Suppressed comments (3)

.github/workflows/build-preview.yml:72

  • The steps: list items in this job are not indented under the steps: key, which makes the workflow YAML invalid and the job will fail to load.
    steps:
    - name: Configure AWS credentials
      uses: aws-actions/configure-aws-credentials@e6de054238d6b7531b4efff3b6587d9aade6a06c #v6.2.3

.github/workflows/build-preview.yml:28

  • Without a job-level condition, this job will run even when the workflow is invoked for cleanup. Gate it on the workflow_dispatch action input so only the intended job runs.
  deploy-preview:
    # Fork PRs don't get AWS credentials, so skip them explicitly rather than let the job fail
    # if: github.event.action != 'closed' && github.event.pull_request.head.repo.full_name == github.repository
    runs-on: ubuntu-latest

.github/workflows/build-preview.yml:69

  • This job currently runs on every workflow_dispatch invocation. Gate it on the workflow_dispatch action input so cleanup only runs when explicitly requested.
  cleanup-preview:
    # if: github.event.action == 'closed' && github.event.pull_request.head.repo.full_name == github.repository
    runs-on: ubuntu-latest

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/build-preview.yml Outdated
Comment thread .github/workflows/build-preview.yml
Comment thread .github/workflows/build-preview.yml
Comment thread .github/workflows/build-preview.yml
Comment thread .github/workflows/build-preview.yml Outdated
Comment thread .github/workflows/build-preview.yml Outdated
Comment thread .github/workflows/build-main.yml Outdated
Comment thread .github/workflows/build-main.yml Outdated
@toblea
toblea merged commit ea1a79b into master Aug 18, 2026
6 of 10 checks passed
@toblea
toblea deleted the DAGE-1226_NewHosting branch August 18, 2026 11:26
@toblea
toblea restored the DAGE-1226_NewHosting branch August 18, 2026 11:26
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