Home Original page

Fix: Correct COPY command syntax in frontend production Dockerfile by trillium · Pull Request #2086 · hackforla/VRMS

Fixed two build errors preventing the frontend Docker image from building:

1. Removed duplicate 'display' key in loadingStyle object (ManageProjects.jsx)
   - The loadingStyle object had both "display": "none" and "display": "flex"
   - Kept "display": "flex" which is correct for the loading overlay
   - Introduced in commit 3a25f7f by JamesNg on 2025-01-29

2. Replaced non-existent StyledButton import with MUI Button (buttonGroup.jsx)
   - Component was importing StyledButton from ProjectForm.jsx
   - StyledButton export was removed in commit 8e9ac7b by Kaipher on 2025-10-06
   - Component was created in commit cf194d2 by JamesNg on 2025-09-29
   - Updated to use standard MUI Button component with matching styles

Build errors resolved:
- "Duplicate key 'display' in object literal"
- "StyledButton is not exported by src/components/ProjectForm.jsx"

Generated with [Claude Code](https://claude.ai/code)
via [Happy](https://happy.engineering)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>

trillium added a commit that referenced this pull request

Feb 15, 2026 
Fixed two syntax errors in the frontend production Dockerfile that were masked by earlier build failures:

1. Line 6: COPY command syntax error
   - Changed: COPY --chown=node:node package.json package.json ./
   - To: COPY --chown=node:node package*.json ./
   - The original attempted to copy package.json twice (invalid syntax)
   - Now uses glob pattern to properly copy package.json and package-lock.json if present
   - Matches the pattern used in backend/Dockerfile.prod:5

2. Lines 15-16: Missing equals signs in ARG and ENV directives
   - Changed: ARG CUSTOM_REQUEST_HEADER nAb3kY-S%qE#4!d
   - To: ARG CUSTOM_REQUEST_HEADER=nAb3kY-S%qE#4!d
   - Changed: ENV REACT_APP_CUSTOM_REQUEST_HEADER $CUSTOM_REQUEST_HEADER
   - To: ENV REACT_APP_CUSTOM_REQUEST_HEADER=$CUSTOM_REQUEST_HEADER
   - Ensures proper Dockerfile syntax compliance

These errors were present in the original file but were not caught because the build was failing earlier due to code issues (duplicate object keys and missing exports) that have been fixed in PR #2086.

Generated with [Claude Code](https://claude.ai/code)
via [Happy](https://happy.engineering)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>

kkchu791

geolunalg

@trillium