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

mrc-6200 Select dataset and admin level on test page #8

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

EmmaLRussell
Copy link
Contributor

@EmmaLRussell EmmaLRussell commented Jan 30, 2025

This branch adds functionality to the test page to allow selecting dataset from both localhost (if running locally) and mrcdata, so you can eyeball a grout tile dataset to see if it actually works. You can also toggle admin1 or admin2 level.

I've added a simple Vue app, which on mount fetches metadata from both servers, and adds datasets to a select (options are prefixed with 'local' or 'mrcdata'). There are radio buttons to select admin level 1 or 2 - admin levels are available in the metadata but for now we're assuming that admin1 and admin2 are defined. admin0 is still used to draw country boundaries.

This means that this page should now be useful both for development testing of the grout server code, and also to check that datasets are working. So it's probably going to be useful to add it to the grout deploy itself so it can be used for testing without needing to have the grout repo checked out. I'll do that in a separate grout deploy ticket.

Just open the page in the browser. You should see that mrcdata now has "arbomap" and "gadm41" datasets.

If this page gets much more complicated, we should probably convert it into a built Typescript app, but for now it's just plain old HTML + js...

@EmmaLRussell EmmaLRussell marked this pull request as ready for review January 30, 2025 12:18
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