Initial setup of project with html and code layout - #11
Conversation
jnrom-dot
left a comment
There was a problem hiding this comment.
Everything is looking functional and seemingly alright, may expect some surprises along the way
| <p className="eyebrow">Campus directions</p> | ||
| <h1 id="page-title">Find your way to class.</h1> | ||
| <p className="intro"> | ||
| Enter your class schedule to get clear directions between campus |
There was a problem hiding this comment.
Should they only put in what classes they need to go in, like will they determine when they leave for class
There was a problem hiding this comment.
The current MVP is that the user will enter one class they want to go to and then be directed to the class room, I believe.
| buildings. | ||
| </p> | ||
| <div className="setup-note" role="status"> | ||
| Your ClassFinder workspace is ready for the class-entry experience. |
There was a problem hiding this comment.
What is the limit on giving all the class names, like one at a time or all of them
There was a problem hiding this comment.
One at a time, for now. But hopefully the user will be able to put in all of their classes in the future.
| @@ -0,0 +1,16 @@ | |||
| import { StrictMode } from 'react'; | |||
There was a problem hiding this comment.
Strict mode is a page where nothing is working have something like a Warning/Reminder that the app is currently unavailable
|
|
||
| - `infrastructure_plan.md` — selected infrastructure decisions. | ||
| - `package.json` — npm scripts and development dependencies. | ||
| - `Dockerfile`, `compose.yml`, `.dockerignore` — reproducible Node.js development environment. |
There was a problem hiding this comment.
Do we need an actions.yml, or build.yml
There was a problem hiding this comment.
I don't believe so, I think the original setup Codex gave us should be satisfactory for now.
|
Got everything working, runs properly on Docker and the webpage loads as expected. |
Mernon
left a comment
There was a problem hiding this comment.
The Docker image loads correctly and the webpage opens and displays correctly.
Before merging, pull to own computer, try running with docker, and check that the web page is reachable.