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

Refine Share #68

Closed
penrods opened this issue Mar 16, 2020 · 6 comments · Fixed by #174
Closed

Refine Share #68

penrods opened this issue Mar 16, 2020 · 6 comments · Fixed by #174
Labels
good first issue Good for newcomers

Comments

@penrods
Copy link
Contributor

penrods commented Mar 16, 2020

The basic Share functionality works, but it could use some refinement. Specfically two things:

  • Improve the filename. Currently it is ".null". Change it to be "PrivateKit_.json"
  • Add a message body. Try:
    Here is my location log from Private Kit.
  • If titles are supported, make it: "Shared PrivateKit_.json"
@penrods penrods added the good first issue Good for newcomers label Mar 16, 2020
@tushar-dadlani
Copy link
Contributor

.null is on android on iOS it shows up as (null)-someid.data

improved the filename for android, need someone to test, for iOS - will need to do something like https://github.com/react-native-community/react-native-share#share-remote-pdf-file-with-gmail--whatsapp-ios

I have made some changes here: https://github.com/tushar-dadlani/private-kit/tree/tdadlani/fix_share_name

would love your thoughts before I open a PR

@tremblerz
Copy link
Contributor

Looks good based on changes, haven't tested it yet. Can you open a PR @tushar-dadlani ?

@tushar-dadlani
Copy link
Contributor

tushar-dadlani commented Mar 23, 2020

I am breaking this into 3 parts

  1. Save file to disk with the correct name
  2. Share the file from filesystem
  3. delete the file locally

@tremblerz
Copy link
Contributor

I would recommend avoiding that route if possible because other Apps can pickup for the time it is on disk (making the data non-private in a sense). At the same time I am aware of this issue being the library issue not ours so take your call.

Private Kit Kanban automation moved this from Backlog to Done Mar 24, 2020
@tremblerz tremblerz reopened this Mar 28, 2020
Private Kit Kanban automation moved this from Done to Dev In progress Mar 28, 2020
@tremblerz
Copy link
Contributor

Export on iOS is still an issue when sent as an attachment on Apps like gmail and whatsapp

@tremblerz
Copy link
Contributor

Fixed in #283

Private Kit Kanban automation moved this from Dev In progress to Done Apr 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants