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

Added Indigenous Lands **Treaties** Layer. Indigenous Lands Treaties Inventory #78

Merged
merged 3 commits into from
Nov 14, 2018

Conversation

kevinzluo
Copy link
Contributor

Fixes #55
3 of 3 PRs

Implemented the Indigenous Lands Treaties Layer

Works similar to #76, #77 .

I have decided to get data only for the specific section rather than the whole map because the territories and treaties layers are quite more complex than the languages ones. I have set clearOutsideBounds to false for now.

showingiltreatiesworkingregionalfalse

Please tell me if you would like me to change anything.

Happy to be contributing,
Kevin Luo

@kevinzluo kevinzluo changed the title Added Indigenous Lands **Treaties** Layer Added Indigenous Lands **Treaties** Layer. Indigenous Lands Treaties Inventory Nov 3, 2018
@jywarren
Copy link
Member

jywarren commented Nov 9, 2018

I merged one and it looks awesome! But that seems to have made a conflict here. Can u resolve?

Thanks!!!

@kevinzluo
Copy link
Contributor Author

I'll take a look as soon as I can! I am currently out of town.

@jywarren
Copy link
Member

jywarren commented Nov 9, 2018 via email

@kevinzluo
Copy link
Contributor Author

@jywarren This one is fixed now too.

Also, should the activity in this repository be showing on Gitter? I see that most of the gitter activity feed is from plots2, but sometimes I also see other repositories, such as spectraworkbench.js, but never this one.

@jywarren
Copy link
Member

Oh excellent. Good point on gitter... i'll check the settings!!! 👍 👍 👍

@jywarren jywarren merged commit 496a82d into publiclab:master Nov 14, 2018
@ghost ghost removed the ready label Nov 14, 2018
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.

3 participants