-
Notifications
You must be signed in to change notification settings - Fork 4
Deploy website using GitHub pages #220
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.
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
Open
Twissi
wants to merge
17
commits into
OpenTechSchool:master
Choose a base branch
from
Twissi:github-pages
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Remove German language support, keep only English - Configure Next.js for static export with GitHub Pages basePath support - Add asset path utility function for proper GitHub Pages URL handling - Update all hardcoded asset paths to use dynamic basePath - Fix font URLs to work with GitHub Pages - Add .nojekyll file to prevent GitHub Pages from ignoring _next files - Update package.json with export script - Remove German translation files and city data - Format code with prettier 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Remove Vercel-specific GitHub Actions workflow - Add GitHub Pages deployment workflow with proper Next.js static export - Configure automatic deployment on push to master/main branches - Use GitHub's official actions for Pages deployment 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Add ignore rules for styled-jsx properties (jsx, global) - Disable explicit-module-boundary-types warnings - Fix React version detection in ESLint settings - Resolves dev server startup issues with linting errors 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Include github-pages-new branch in GitHub Pages workflow triggers - Allows testing deployment from feature branch before merging 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Update actions/upload-pages-artifact from v2 to v3 - Update actions/deploy-pages from v2 to v4 - Fixes deprecated action version errors 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Add 'output: export' to next.config.js - Ensures out/ directory is generated during build - Fixes GitHub Actions artifact creation error 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Remove output: 'export' config (Next.js 12 doesn't support it) - Update export script to run 'next build && next export' - Convert getInitialProps to getStaticProps/getStaticPaths for cities - Remove getInitialProps from withLocale container and _document - Fix data serialization in cities getStaticProps - Update GitHub Actions workflow to use export command - Successfully generates static files in out/ directory 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Update all remaining hardcoded asset paths to use assetPath utility for proper GitHub Pages basePath support: - CityHero component background images - Meta component favicons and manifest links - MarkdownTeam component member images - Cities page social icons and about images 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Update global CSS font declarations to use dynamic basePath for GitHub Pages deployment: - Add environment detection for GitHub Actions - Use GITHUB_REPOSITORY to extract repository name for basePath - Apply basePath to OpenSans, Roboto, and Material Icons font URLs - Ensures fonts load correctly when deployed with basePath on GitHub Pages 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Replace outdated Zeit.co deployment information - Add detailed instructions for building with GitHub Pages basePath locally - Include environment variables needed for proper GitHub Pages build - Explain difference between local and GitHub Pages builds - Document asset path behavior with basePath configuration 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Remove Slack channel mentions from contribution guidelines - Replace Slack communication with GitHub issue discussions - Encourage using GitHub issues for help and collaboration - Simplify communication channels to focus on GitHub-based workflow 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Add font preloading to _document.tsx to ensure fonts are cached during initial page load, preventing FOUT during client-side navigation. The preload links use environment-aware basePath detection for proper GitHub Pages deployment. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Code generate with Claude.
Also pushed directly to
github-pages
branch in this repo to see the deployment working: https://opentechschool.github.io/websitenext/After merging this we need to:
You can ignore the deploy action/PR preview on this PR since it will be deleted once this PR get's merged.
===============================================
Convert Next.js app to static GitHub Pages deployment
This PR converts the OpenTechSchool website from a dynamic Next.js app to a static site that can be deployed on GitHub Pages, while removing multi-language support to keep only English.
🎯 Key Changes
Static Site Conversion:
Asset Path Management:
Multi-language Removal:
Font Loading Optimization:
Deployment Infrastructure:
Code Quality:
🔧 Technical Details
🧪 Testing
🤖 Generated with https://claude.ai/code