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

Multiple Merge Pulls & update to Cloud Cover URL #324

Merged
merged 11 commits into from
Nov 10, 2017

Conversation

nfspeedypur
Copy link
Contributor

The only change is the update to the Cloud Cover URL. However this does include the merge pull requests since the last pull back to master.

-David

Copy link
Collaborator

@rladines rladines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The URL should have "map" not "maps".

@nfspeedypur
Copy link
Contributor Author

I have updated the URL, based on your feedback

@rladines rladines assigned tomschenkjr and unassigned rladines Nov 9, 2017
@rladines
Copy link
Collaborator

rladines commented Nov 9, 2017

@tomschenkjr I am done with my review and approved the changes. You can go ahead and merge now into Chicago:dev. I am not authorized to do this for this PR.

@tomschenkjr tomschenkjr merged commit 9c3f79f into Chicago:master Nov 10, 2017
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