Conversation
- Add GitHub Actions workflow for automated deployment to GitHub Pages - Configure dual-branch deployment: main (/) and dev (/dev/) - Update vite.config.ts to support dynamic base paths via VITE_BASE_PATH - Add deployment documentation with setup and troubleshooting guides - Update amenities page with new images: - Vertical farm with autonomous robots for Grass Lab section - Tennis court with biomechanics HUD for Multi-Sport Ecosystem section - Improve image alt text for better accessibility 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove npm cache configuration that requires lock file - Change from npm ci to npm install for more flexible dependency resolution - Add .gitignore to exclude node_modules and build artifacts - Add package-lock.json for reproducible builds and faster CI installs This fixes the deployment workflow error: "Dependencies lock file is not found" 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add modern, professional README structure with emojis and clear sections - Include complete facility layout and technology stack details - Document development setup, deployment process, and project architecture - Add links to live demo, dev preview, and deployment guide - Integrate information from project requirements document - Include strategic goals, partnerships, and technical specifications - Add project structure diagram and configuration examples - Maintain all original technical details from requirements This creates a developer-friendly, investor-ready README that clearly communicates the project's vision, technology, and value proposition. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove non-existent /index.css link from index.html that was breaking dev build - Add clean step in workflow to ensure fresh build for dev branch - This fixes the blank page issue on https://kvnloo.github.io/ace/dev/ Root cause: The index.html referenced /index.css which doesn't exist. During build with VITE_BASE_PATH=/dev/, Vite couldn't resolve this path correctly, causing the dev deployment to fail silently with a blank page. Solution: Remove the unused CSS link since all styles are inline or handled by Tailwind CDN. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
ROOT CAUSE: GitHub Pages serves the repo at /ace/, not at domain root. The base paths were incorrectly set to / and /dev/ instead of /ace/ and /ace/dev/. This caused: - Main: Scripts loading from /assets/ instead of /ace/assets/ - Dev: Scripts loading from /dev/assets/ instead of /ace/dev/assets/ - Result: 404 errors, blank pages CHANGES: - Update main build: VITE_BASE_PATH='/ace/' (was '/') - Update dev build: VITE_BASE_PATH='/ace/dev/' (was '/dev/') This ensures Vite generates correct absolute paths: - Main: /ace/assets/index-xxx.js ✅ - Dev: /ace/dev/assets/index-xxx.js ✅ After deployment, pages will load correctly: - https://kvnloo.github.io/ace/ (main) - https://kvnloo.github.io/ace/dev/ (dev) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
/, dev →/dev/)Changes
GitHub Actions Workflow
.github/workflows/deploy.ymlfor automated deploymentsmainanddevbranches on every push to either branch/dev/subdirectoryConfiguration Updates
vite.config.tsto support dynamic base pathsVITE_BASE_PATHenvironment variable for build-time path configurationAmenities Page Enhancements
Documentation
.github/DEPLOYMENT.mdwith:Testing Checklist
https://kvnloo.github.io/ace/https://kvnloo.github.io/ace/dev/Deployment URLs
After merging and enabling GitHub Pages:
🤖 Generated with Claude Code