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

Fixed broken Mapbox logo link in Readme #174

Closed
wants to merge 1 commit into from
Closed

Conversation

pratikyadav
Copy link
Contributor

No description provided.

@akacarlyann
Copy link
Collaborator

akacarlyann commented Dec 13, 2019

Thanks! This is a duplicate of #166 but may point to a more reliable image.

@sgillies sgillies closed this Dec 13, 2019
@sgillies sgillies reopened this Dec 13, 2019
@sgillies sgillies removed their request for review December 13, 2019 23:59
@pratikyadav
Copy link
Contributor Author

@akacarlyann Thanks for pointing it out. :)

Yes, it is a duplicate but #166 was out of date with the master branch to merge which I saw after opening this PR.

@akacarlyann akacarlyann requested review from akacarlyann and removed request for akacarlyann December 14, 2019 00:17
@akacarlyann
Copy link
Collaborator

@pratikyadav Ah I see. The maintainer can merge master into the dev branch on #166 to bring it up to date. Are you referencing the same logo file?

@pratikyadav
Copy link
Contributor Author

Are you referencing the same logo file?

No, not exactly the same in terms of dimension.

@pratikyadav pratikyadav deleted the readme-logo branch April 19, 2021 05:03
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.

None yet

3 participants