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

Add some missing props to TypeScript lib def #1360

Merged
merged 2 commits into from
Jun 4, 2021

Conversation

draperunner
Copy link
Contributor

@draperunner draperunner commented May 21, 2021

Description

This adds some missing props to the TypeScript lib def file that I have encountered.

I am aware of #1067, but I guess it will still take some time before that one is merged and released.

I have not used onImageMissing myself, but from looking at the source code it seems like this function is called with the name of the missing image as a string. Please correct me if I'm wrong.

Checklist

  • I have tested this on a device/simulator for each compatible OS
  • I updated the documentation yarn generate
  • I mentioned this change in CHANGELOG.md
  • I updated the typings files (index.d.ts)
  • I added/ updated a sample (/example)

@ferdicus
Copy link
Member

ferdicus commented Jun 4, 2021

Thanks for your contribution 🙇🏿

@ferdicus ferdicus merged commit f97925c into rnmapbox:master Jun 4, 2021
@draperunner draperunner deleted the add-missing-ts-props branch July 5, 2021 15:17
mikalaiulasevich pushed a commit to OneSoil-Platform/maps that referenced this pull request Sep 23, 2021
* Add some missing props to TypeScript lib def

* Add changelog entry
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