Skip to content

feat: migrate deployment and previews on Github Pages #227

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.

Sign up for GitHub

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

Jump to bottom
Merged
bengeois merged 4 commits into from
Dec 25, 2025
Merged

feat: migrate deployment and previews on Github Pages #227

bengeois merged 4 commits into from
Dec 25, 2025

Conversation

@bengeois
Copy link
Collaborator

@bengeois bengeois commented Dec 20, 2025 •
edited by Gudsfile
Loading

1️⃣ First

🔇 Problem

The project was using Firebase Hosting for deployment, which required secrets and external infrastructure. Additionally, the blog was deployed separately on Github Pages, creating fragmentation in the deployment process.

🎹 Proposal

Complete migration to GitHub Pages for deployment:

Commit 1: Deploy blog on Github Pages (5cfb4b4)

  • Simplified blog deployment workflow
  • Using GitHub Pages for blog hosting

Commit 2: Deploy app on Github Pages (58a4542)

  • Migrated application deployment to GitHub Pages
  • Configured Astro with base: '/tracksy' and site: 'https://gudsfile.github.io'

Commit 3: Deploy PR preview on Github Pages (736cd7a)

  • Dynamic base path configuration via process.env.BASE_PATH || '/tracksy' in astro.config.mjs
  • Adapted preview workflow to support dynamic paths (/tracksy/pr-preview/pr-XXX/)
  • Enables functional previews for each PR on GitHub Pages

🎶 Comments

This migration brings several benefits:

  • ✅ Removed dependency on Firebase Hosting
  • ✅ Consolidated deployment on a single platform (GitHub Pages)
  • ✅ Automatic and functional PR previews

🎤 Test

Tested Configuration

  1. Blog: Deployed on GitHub Pages and accessible at https://gudsfile.github.io/tracksy/blog/
  2. App in production: Deployed at https://gudsfile.github.io/tracksy/ with correct base path
  3. PR previews: Accessible at https://gudsfile.github.io/tracksy/pr-preview/pr-XXX/ with resources loading correctly

Verifications

  • Static assets load correctly with the new base path
  • Navigation works on the production app
  • PR previews have a functional dynamic base path
  • Blog is accessible

Additional Steps

  • Delete Firebase secrets from the repository (FIREBASE_SERVICE_ACCOUNT, etc.) (@Gudsfile)
  • Remove project on Firebase workspace (@bengeois)

Closes #221

@bengeois bengeois self-assigned this Dec 20, 2025
@github-actions
Copy link

github-actions bot commented Dec 21, 2025 •
edited
Loading

PR Preview Action v1.7.0
Preview removed because the pull request was closed.
2025-12-25 19:39 UTC

@bengeois bengeois added enhancement New feature or request ci/cd Impacts continuous integration and continuous delivery [ [ meta ] ] Refers to project management, organizational decisions… and removed preview labels Dec 21, 2025
@bengeois bengeois marked this pull request as ready for review December 21, 2025 20:20
@bengeois bengeois changed the title chore: update GitHub Actions workflow for blog to deploy feat: migrate deployment and previews on Github Pages Dec 21, 2025
@bengeois bengeois force-pushed the ft/deploy-gh-pages branch 4 times, most recently from fc08017 to c300484 Compare December 22, 2025 13:47
Tarektouati
Tarektouati reviewed Dec 22, 2025
View reviewed changes
.github/workflows/blog-deploy-live.yml Outdated
Comment on lines 6 to 8
paths:
- "blog/**"
- ".github/workflows/blog.yml"
Copy link
Collaborator

@Tarektouati Tarektouati Dec 22, 2025

Choose a reason for hiding this comment

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

Maybe we should include also the .moon/toolchain.yml and .moon/workspace.yml as well
In this case:
If we bump golang version in the tool chain or moon related stuff, it should re-deploy the blog

Copy link
Collaborator Author

@bengeois bengeois Dec 22, 2025

Choose a reason for hiding this comment

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

I've only added the .moon/toolchains.yml file. I don't see any value to add .moon/workspace.yml for now as it is only for repository configuration

.github/workflows/app-deploy-live.yml
Comment on lines +6 to +8
paths:
- "app/**"
- ".github/workflows/app-deploy-live.yml"
Copy link
Collaborator

@Tarektouati Tarektouati Dec 22, 2025

Choose a reason for hiding this comment

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

Maybe we should include also the .moon/toolchain.yml and .moon/workspace.yml as well
In this case:
If we bump node.js version in the tool chain or moon related stuff, it should re-deploy the application

Copy link
Collaborator Author

@bengeois bengeois Dec 22, 2025

Choose a reason for hiding this comment

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

I've only added the .moon/toolchains.yml file. I don't see any value to add .moon/workspace.yml for now as it is only for repository configuration

@bengeois bengeois force-pushed the ft/deploy-gh-pages branch 2 times, most recently from 6872808 to 8282079 Compare December 22, 2025 22:39
@bengeois bengeois requested a review from Tarektouati December 22, 2025 22:40
Gudsfile
Gudsfile reviewed Dec 22, 2025
View reviewed changes
Gudsfile
Gudsfile previously approved these changes Dec 22, 2025
View reviewed changes
Gudsfile
Gudsfile previously approved these changes Dec 23, 2025
View reviewed changes
@bengeois bengeois merged commit 162ef44 into main Dec 25, 2025
7 checks passed
@bengeois bengeois deleted the ft/deploy-gh-pages branch December 25, 2025 19:38
github-actions bot pushed a commit that referenced this pull request Dec 25, 2025
@bengeois
chore(): remove Tracksy App Preview for PR #227
88c3f55
@bengeois
Copy link
Collaborator Author

bengeois commented Dec 25, 2025 •
edited
Loading

@Gudsfile Can you remove unused secrets from Repository Settings as we're now on Github Pages?
You can also update the link in the repository description 🎉

@Gudsfile
Copy link
Owner

Gudsfile commented Dec 26, 2025 •
edited
Loading

@Gudsfile Can you remove unused secrets from Repository Settings as we're now on Github Pages?

You can also update the link in the repository description 🎉

Done 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Gudsfile Gudsfile Gudsfile approved these changes

@Tarektouati Tarektouati Awaiting requested review from Tarektouati Tarektouati is a code owner

Assignees

@bengeois bengeois

Labels

ci/cd Impacts continuous integration and continuous delivery enhancement New feature or request [ [ meta ] ] Refers to project management, organizational decisions…

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Move deployment to Github Pages

4 participants

@bengeois @Gudsfile @Tarektouati