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

Update Argentina resources extended descriptions #92

Merged
merged 7 commits into from
Apr 18, 2018

Conversation

ignaciolep
Copy link
Contributor

Improved extended descriptions (now that we know how are displayed)

PS: Still trying to learn how GitHub works, tell me if I done something wrong. (I synced my fork comparing with master and creating a pull request, then updated resource files and created this pull request.)

Updated extended description (now that we know how is displayed)
Updated extended description (now that we know how is displayed)
Updated extended description (now that we know how is displayed)
Updated extended description (now that we know how is displayed)
Updated extended description (now that we know how is displayed)
Updated extended description (now that we know how is displayed)
@bhousel
Copy link
Member

bhousel commented Apr 18, 2018

Improved extended descriptions (now that we know how are displayed)

This looks really great thank you! I noticed that you said in the description that the mailing list is no longer used. I wonder if we should add a special property for something like this or whether we should encourage communities to remove unused resources. (We don't have to decide now).

PS: Still trying to learn how GitHub works, tell me if I done something wrong. (I synced my fork comparing with master and creating a pull request, then updated resource files and created this pull request.)

This is perfect, you've got it 👍

@bhousel bhousel merged commit 05f8137 into osmlab:master Apr 18, 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.

2 participants