Uh oh!
There was an error while loading. Please reload this page.
[Android] Make ReactWebView & Client protected - #10105
Conversation
In order to more easily extend ReactWebViewManager logic
ghost
commented
Sep 26, 2016
By analyzing the blame information on this pull request, we identified @sathyapriya-31 and @jvassbo to be potential reviewers. |
ghost
commented
Sep 26, 2016
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks! |
ghost
commented
Sep 26, 2016
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
facebook-github-bot
commented
Oct 27, 2016
It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @jacobp100 as a potential reviewer. Could you take a look please or cc someone with more context? |
cbrevik
commented
Oct 28, 2016
CC @satya164 |
facebook-github-bot
commented
Nov 11, 2016
@hramos has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
I'd like to make
ReactWebViewandReactWebViewClientprotectedinstead ofprivate, inside theReactWebViewManagerclass. The reason being that if you extend theReactWebViewManagerit'll be much easier to override existing logic.In my specific case I'd like to be able to override shouldOverrideUrlLoading inside
ReactWebViewClientand handle custom URL schemes differently. See #10055.