Skip to content

create dynamic map#18

Open
owtmelk wants to merge 1 commit intodatamade:mainfrom
owtmelk:map-features
Open

create dynamic map#18
owtmelk wants to merge 1 commit intodatamade:mainfrom
owtmelk:map-features

Conversation

@owtmelk
Copy link
Copy Markdown

@owtmelk owtmelk commented Mar 26, 2026

Overview

Displays Chicago's community areas and shades them depending on how many new restaurant permits were issued in a given year. Adds some descriptive capabilities for each community on hover.

Notes

When I was working on the challenge I debated having multiple useEffect() calls to handle state updates for the total permit numbers. Mainly because I was learning react and confused. I went with one since everything has the same dependency on the data endpoint.

I haven't used React much, what are best practices around useEffect()? Could we calculate everything during rendering in this case? Might look at that.

reference react docs - https://react.dev/learn/you-might-not-need-an-effect

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.

1 participant