Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated Input1 Page #131

Merged
merged 3 commits into from
Jan 10, 2024
Merged

Updated Input1 Page #131

merged 3 commits into from
Jan 10, 2024

Conversation

clayton-schneider
Copy link
Collaborator

  • Swapped over to tailwind styles
  • Inputs are now ShadCN based
  • Wrapped in Remix Form
  • Created a form handler at the /homes route to handle the submission from the form on /inputs1

@jkoren
Copy link
Collaborator

jkoren commented Jan 7, 2024

Hi @clayton-schneider - its great that you added this code.
I wanted to test it out, so I cloned your repo and installed.

I did this with

git clone https://github.com/clayton-schneider/home-energy-analysis-tool.git
cd ./home-energy-analysis-tool/heat-stack
npm install 
cp .env.example .env
git checkout dev
npm run dev

It looks nice!
Let me know if there is an easier way to test the code.

image

@thadk thadk merged commit d176a35 into codeforboston:dev Jan 10, 2024
8 checks passed
thadk pushed a commit that referenced this pull request Jan 11, 2024
* added baseUrl, swapped styles to tw, swapped to shad

* wrapped in form and added route to handle form action

* spell checked
AdamFinkle pushed a commit to AdamFinkle/home-energy-analysis-tool that referenced this pull request May 22, 2024
* added baseUrl, swapped styles to tw, swapped to shad

* wrapped in form and added route to handle form action

* spell checked
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.

3 participants