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.

Sign up for GitHub

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

Jump to bottom

Calculator #23

Merged
merged 8 commits into from
Mar 15, 2024
Merged

Calculator #23

merged 8 commits into from
Mar 15, 2024

Conversation

lishaduck
Copy link
Member

@lishaduck lishaduck commented Mar 1, 2024 •
edited
Loading

Description

This PR adds the price calculator.

Type of Change

  • ✨ New feature

Checklist

  • Read the Contributing Guidelines.
  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Fill out this template.
  • Log your hours.
  • Check that commits follow the Angular commit convention, more or less.
  • Ideally, include relevant tests that fail without this PR but pass with it (if applicable).

Tested on

  • macOS 14

@lishaduck lishaduck force-pushed the main branch 5 times, most recently from feb5d57 to fb3ba3c Compare March 3, 2024 20:47
@lishaduck lishaduck marked this pull request as ready for review March 5, 2024 19:21
MattsAttack
MattsAttack previously approved these changes Mar 11, 2024
View reviewed changes
lishaduck and others added 7 commits March 14, 2024 20:22
@lishaduck @MattsAttack
feat: add solar pricing calculator
8d095b0 
* Add Tailwind form reset and the headlessui plugin
* Add a new page for the solar pricing calculator
* Add a new island for comboboxes
* Add a TODO comment about testing ;)
* Add a new utility for grabbing geo data from IP addresses

Co-authored-by: Matthew Wasser <71449347+MattsAttack@users.noreply.github.com>
@lishaduck
chore: minor tweaks
01f6802 
* Fix flexboxes
* Center page content by default
* Clarify vendored tailwind plugin by renaming to `headless-tailwind.ts`
@lishaduck
style: deno.json sorting
c587144
@lishaduck
refactor: remove redundancy
b2bba61
@lishaduck
fix: add an expected animation
190dc46
@lishaduck
chore: update copy
ea9dcce
@lishaduck
fix: offline
e928e61 
Now, rather than crashing, there's just no default.
@lishaduck
fix: literally everything about the calculator
9194deb 
* CSS
* Logic
* Everything else
@lishaduck lishaduck merged commit 83d4d48 into main Mar 15, 2024
5 checks passed
@lishaduck lishaduck deleted the calc branch March 15, 2024 01:41
@lishaduck lishaduck added this to the State milestone Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MattsAttack MattsAttack MattsAttack left review comments

@iiAzui iiAzui Awaiting requested review from iiAzui iiAzui is a code owner

Assignees

@MattsAttack MattsAttack

@lishaduck lishaduck

Labels
None yet
Projects
None yet
Milestone
State
Development

Successfully merging this pull request may close these issues.
2 participants
@lishaduck @MattsAttack