Skip to content

Add method to open native share sheet from within web-view.#61

Merged
fdocr merged 1 commit intoforem:masterfrom
VarunBarad:feature/native-share-sheet
Aug 4, 2020
Merged

Add method to open native share sheet from within web-view.#61
fdocr merged 1 commit intoforem:masterfrom
VarunBarad:feature/native-share-sheet

Conversation

@VarunBarad
Copy link
Copy Markdown
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

This PR adds the native method required to show the Android share sheet when sharing text from the website inside the web-view.

To complete the integration, some code needs to be added to the https://dev.to also. Will create an issue for that over on thepracticaldev/dev.to with specific instructions as to what changes need to be made.

Related Tickets & Documents

Integrating this and its sibling changes in thepracticaldev/dev.to (issue not created yet) will fix #11

Comment thread app/src/main/java/to/dev/dev_android/util/AndroidWebViewBridge.kt
Copy link
Copy Markdown
Contributor

@fdocr fdocr left a comment

Choose a reason for hiding this comment

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

As mentioned in my last comment from #11 I left a suggestion inline. Thank you again for the contribution @VarunBarad!

@VarunBarad VarunBarad force-pushed the feature/native-share-sheet branch from b9cd514 to 7c63b0b Compare August 2, 2020 18:55
@VarunBarad
Copy link
Copy Markdown
Contributor Author

VarunBarad commented Aug 2, 2020

@fdoxyz I have rebased this branch from current master branch on thepracticaldev/DEV-Android.

Will you be needing anything else now?

@VarunBarad VarunBarad requested a review from fdocr August 2, 2020 18:59
Copy link
Copy Markdown
Contributor

@fdocr fdocr left a comment

Choose a reason for hiding this comment

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

This looks great @VarunBarad thank you again for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web Share API not working

2 participants