Skip to content

Commit

Permalink
Merge pull request #11 from dxe/alex/lowercase
Browse files Browse the repository at this point in the history
lREADME: mention filenames should be lowercase
  • Loading branch information
alexsapps authored Jan 29, 2025
2 parents 4f9462c + 2eecfe9 commit 8023a5b
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,12 @@ You can edit the pages by modifying `pages/*.tsx`. Your browser auto-updates as

## Development

New pages may need to be added to the `PAGES_TO_ALWAYS_SHRINK_NAVBAR` array for proper formatting.
**Page filenames must be lowercase.** In the past we experienced discrepancies
with case sensitivity between S3 and our app's routing, so we configured S3 to
invoke our custom 404 page, and that page to rewrite all URLs to lowercase.

New pages may need to be added to the `PAGES_TO_ALWAYS_SHRINK_NAVBAR` array for
proper formatting.

The use of Bootstrap for CSS in this project is deprecated. (Tailwind for CSS might be cool.)

Expand All @@ -29,4 +34,6 @@ makes use of `launch.json`.

## Deployment

A GitHub Action automatically deploys the website to http://righttorescue.com after pushing to main.
A GitHub Action automatically deploys the website to http://righttorescue.com
after pushing to the main branch. Next.js generates static files for this
website, and these are pushed to AWS S3.

0 comments on commit 8023a5b

Please sign in to comment.