Skip to content

revert: remove gh-pages deploy, keeping original Pages setup#18

Merged
stackingsaunter merged 1 commit intogetAlby:mainfrom
hermes-alby:revert/gh-pages-deploy
Apr 16, 2026
Merged

revert: remove gh-pages deploy, keeping original Pages setup#18
stackingsaunter merged 1 commit intogetAlby:mainfrom
hermes-alby:revert/gh-pages-deploy

Conversation

@hermes-alby
Copy link
Copy Markdown
Contributor

Reverts the changes from #16 that switched deploy-pages.yml to the gh-pages branch approach and added pr-preview.yml.

Instead, we'll use Netlify for PR preview deployments — it requires zero workflow changes, keeps the production GitHub Pages setup untouched, and auto-deploys previews on every PR.

What this does:

  • Restores deploy-pages.yml to the original GitHub Pages API approach
  • Removes pr-preview.yml (not needed with Netlify)
  • Reverts VITE_BASE_URL change in vite.config.ts

Netlify setup (one-time, 5 min):

  1. Create account at netlify.com (free for open source)
  2. Add new site → Import existing project → connect getAlby/bitcoin-apps
  3. Build command: npm run build, Publish directory: dist
  4. Deploy

After that, every PR automatically gets a preview URL posted as a comment.

Going with Netlify for PR previews instead, which requires zero
workflow changes and keeps the production GitHub Pages setup untouched.

Reverts deploy-pages.yml to the original GitHub Pages API approach
and removes pr-preview.yml. Also reverts the VITE_BASE_URL change
in vite.config.ts since it's no longer needed.
@github-actions
Copy link
Copy Markdown
Contributor

PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://getAlby.github.io/bitcoin-apps/pr-preview/pr-18/

Built to branch gh-pages at 2026-04-10 14:11 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@stackingsaunter stackingsaunter merged commit f4e356c into getAlby:main Apr 16, 2026
1 check passed
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