Skip to content

Build and edit in Shakespeare#1

Open
marykatefain wants to merge 5 commits into
NotThatKindOfDrLiz:mainfrom
marykatefain:main
Open

Build and edit in Shakespeare#1
marykatefain wants to merge 5 commits into
NotThatKindOfDrLiz:mainfrom
marykatefain:main

Conversation

@marykatefain
Copy link
Copy Markdown

- Change Vite base from '/bunch/' to './' for relative paths
- Update favicon paths in index.html to be relative
- Simplify 404.html redirect for client-side routing
- Remove BrowserRouter basename (use root path)
- Update redirect handling to use sessionStorage
- Replace BASE_URL reference in logo src with absolute path /logo-name.png
- Simplify customer view button to use absolute path /customer
- Removes undefined prefix issue in image sources
- Add build:github script for GitHub Pages deployment with /bunch/ base
- Update vite.config to use mode-based base path (github mode vs default)
- Create paths utility (getAssetPath, getRoutePath) for environment-aware URLs
- Update MerchantApp to use path utilities for logo and customer view link
- Update 404.html redirect to handle both root and /bunch/ base paths
- Update GitHub Actions workflow to use build:github script

Now supports:
- Shakespeare: base ./ (relative paths, root deployment)
- GitHub Pages: base /bunch/ (absolute paths with prefix)
- Add fallback for undefined BASE_URL
- Use optional chaining for endsWith call
- Prevents "Cannot read properties of undefined" error
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.

1 participant