Skip to content

Initial setup of project with html and code layout - #11

Merged
Mernon merged 1 commit into
mainfrom
initial_setup
Sep 28, 2026
Merged

Mernon merged 1 commit into
mainfrom
initial_setup

Conversation

@IanTLiggett

Copy link
Copy Markdown
Collaborator

Before merging, pull to own computer, try running with docker, and check that the web page is reachable.

@jnrom-dot jnrom-dot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything is looking functional and seemingly alright, may expect some surprises along the way

Comment thread src/App.tsx
<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

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should they only put in what classes they need to go in, like will they determine when they leave for class

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread src/App.tsx
buildings.
</p>
<div className="setup-note" role="status">
Your ClassFinder workspace is ready for the class-entry experience.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the limit on giving all the class names, like one at a time or all of them

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One at a time, for now. But hopefully the user will be able to put in all of their classes in the future.

Comment thread src/main.tsx
@@ -0,0 +1,16 @@
import { StrictMode } from 'react';

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strict mode is a page where nothing is working have something like a Warning/Reminder that the app is currently unavailable

Comment thread README.md

- `infrastructure_plan.md` — selected infrastructure decisions.
- `package.json` — npm scripts and development dependencies.
- `Dockerfile`, `compose.yml`, `.dockerignore` — reproducible Node.js development environment.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need an actions.yml, or build.yml

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't believe so, I think the original setup Codex gave us should be satisfactory for now.

@Mernon

Mernon commented Sep 27, 2026

Copy link
Copy Markdown
Collaborator

Got everything working, runs properly on Docker and the webpage loads as expected.

@Mernon
Mernon self-requested a review September 27, 2026 19:02

@Mernon Mernon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Docker image loads correctly and the webpage opens and displays correctly.

@owenwolfe owenwolfe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Mernon
Mernon merged commit a98b812 into main Sep 28, 2026
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.

4 participants