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

Example of including county data #46

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

sbonds
Copy link

@sbonds sbonds commented Apr 2, 2020

As you can see from the code, I barely know Javascript but was able to create a working example of what I had in mind for issue #44. Ideally instead of "Oregon" appearing in the list of countries, an additional selector would appear when "US" is chosen that would contain each of the 50 states.

However, hopefully this simple-yet-hardcoded example shows what I had in mind for County-specific data.

Apologies for the auto-formatting which obscures the very few real changes.

@sbonds
Copy link
Author

sbonds commented Apr 3, 2020

If merging, watch out for the URL change in createURL() to support my demo site. That should not be merged into the main branch.

@bdarfler bdarfler mentioned this pull request Apr 9, 2020
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