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

[RNMobile] Release v1.6.0 #16108

Closed
wants to merge 1 commit into from
Closed

[RNMobile] Release v1.6.0 #16108

wants to merge 1 commit into from

Conversation

hypest
Copy link
Contributor

@hypest hypest commented Jun 12, 2019

This PR tracks the release v1.6.0 of the native mobile app.

We will not merge this one actually since the code it includes was only meant as a quick-fix until a more proper (and mergeable) fix happened. See #15863 (comment).

In #15551, these were changed to support passing props, but that also means
that when you pass the video icon to the `Icon` component, it won't inject the
right size, since it's not a SVG component.

Instead, we can export the new function to allow for a customizable icon, while
exporting the SVG component by default.
@hypest hypest added the Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change) label Jun 12, 2019
@hypest hypest added this to the 5.8 (Gutenberg) milestone Jun 12, 2019
@hypest
Copy link
Contributor Author

hypest commented Jun 12, 2019

Closing because we won't merge it to master. See PR description above.

The release is tagged as rnmobile/release-v1.6.0 and is safe to delete the branch.

@hypest hypest closed this Jun 12, 2019
@hypest hypest deleted the rnmobile/release-v1.6.0 branch June 12, 2019 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mobile App - i.e. Android or iOS Native mobile impl of the block editor. (Note: used in scripts, ping mobile folks to change)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants