Open
Conversation
…iles, and update animation logic to use config settings for delays.
✅ Deploy Preview for hacklahoma2026 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
This is a deployment PR that removes backend server functionality and adds a new FAQ page with interactive features to the frontend, updates sponsor information, and converts login links to external registration links.
Key Changes:
- Complete removal of backend server code (package.json, models, configuration)
- New interactive FAQ page with passport-style design and selfie capture capability
- Integration of American Fidelity as a new sponsor
- Conversion of login navigation to external registration links with proper security attributes
- Addition of CSS animations for UI enhancements
Reviewed changes
Copilot reviewed 10 out of 25 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| server/package.json | Deleted backend server dependencies configuration |
| server/models/User.js | Removed user data model definition |
| server/.gitignore | Removed backend ignore patterns |
| server/.DS_Store | macOS system file (should be gitignored) |
| frontend/src/ui/pages/landing/sections/Sponsors.tsx | Added American Fidelity sponsor logo and link |
| frontend/src/ui/pages/landing/sections/LandingView.tsx | Updated login links to external registration URL with security attributes and mobile menu styling fix |
| frontend/src/ui/pages/landing/index.tsx | Integrated new FAQ page component into landing page |
| frontend/src/ui/pages/faq/section/SelfieCapture.tsx | New component for capturing user selfies with camera access and IndexedDB storage |
| frontend/src/ui/pages/faq/index.tsx | New interactive FAQ page with passport-style UI and dismissible stamp overlays |
| frontend/src/ui/common/assets/sponsors/AmericanFidelity.png | New sponsor logo image asset |
| frontend/src/ui/common/assets/faq/stamps/stamp_three.png | New stamp graphic asset for FAQ page |
| frontend/src/index.css | Added custom animations (float, spin, float-spin variants) for UI elements |
| frontend/package.json | Added GSAP animation library dependency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
293
to
297
| <a href="https://register.hacklahoma.org/" target="_blank" rel="noopener noreferrer"> | ||
| <button className="px-6 py-3 border-2 border-[#575f49] text-[#575f49] font-medium hover:bg-[#575f49] hover:text-[#F5F5DC] transition-colors duration-300 rounded"> | ||
| Register Now | ||
| </button> | ||
| </a> |
There was a problem hiding this comment.
The buttons inside anchor tags create invalid HTML. Buttons should not be nested inside anchor tags. Instead, style the anchor tag as a button or use onClick handlers on the button with navigation logic.
…ng of the passport
…t' into integration
…t' into deployment
Hotfix for memory leaks when camera is on state. Hotfix for type safety.
…ue IDs for better accessibility and styling.
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
2 participants
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.
Branch Deployment to deploy
Created FAQ
Added new assets
Linked Registration