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

First cut at RI state calculator #15

Closed
wants to merge 21 commits into from
Closed

First cut at RI state calculator #15

wants to merge 21 commits into from

Conversation

oyamauchi
Copy link
Member

Description

  • New .html page for the RI calculator

  • Card-based presentation of incentives, rather than the table like in
    the IRA calculator. This follows the initial designs.

No utility incentives yet, only state + federal. Utility selection
will come in a followup.

If you enter a non-RI zip code, the API gives the error "we don't yet
have state-level coverage in ". This isn't exactly what we want
-- it should say "that zip/address is not in Rhode Island". This will
require an API change to add a "state" parameter.

Test Plan

Enter a RI zip code and make sure that RI incentives come up.

Note this new page will not be available in the Vercel preview. I
haven't configured the Parcel build to include rhode-island.html
yet, because I don't want the page to be publicly available when this
lands.

Next Steps

Follow-up PRs that add the utility and project fields.

@vercel
Copy link

vercel bot commented Aug 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
embed-rewiringamerica-org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 16, 2023 2:26am

@oyamauchi oyamauchi removed the request for review from RandomEtc August 22, 2023 15:21
@oyamauchi oyamauchi marked this pull request as draft August 22, 2023 15:21
@oyamauchi
Copy link
Member Author

Hold the reviews; I'm just going to keep going on the latest design rev. No point reviewing an implementation that's going to be mostly replaced very soon.

This is still missing the icon tabs, which means it's also missing the
selector UI for the "other" incentives.
For the frontend's project selector, we want to combine several of the
API's item types (e.g. `new_electric_vehicle` and
`used_electric_vehicle`) into a single project. This introduces the
abstraction layer to allow that.
This makes an effort to make it read nicely. It's not localizable, but
that's OK for now; RI does not seem to care about having the
calculator available in other languages.
@oyamauchi
Copy link
Member Author

Ditching this in favor of #20.

@oyamauchi oyamauchi closed this Sep 18, 2023
@oyamauchi oyamauchi deleted the owen/ri1 branch September 18, 2023 22:08
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.

2 participants