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

feat: implementing sharing files #29

Merged
merged 4 commits into from
Jun 5, 2024
Merged

feat: implementing sharing files #29

merged 4 commits into from
Jun 5, 2024

Conversation

kayode0x
Copy link
Contributor

@kayode0x kayode0x commented Jun 2, 2024

Summary:

  • Current Functionality: File upload is functional and supports multiple files.
  • Current Limitation: All files are currently processed through kUTTypeURL.

Next Steps:

  • Testing Against kUTTypeFileURL: I am in the process of testing file uploads against kUTTypeFileURL. This will help differentiate between general URLs and file URLs more effectively.

Notes:

  • Plan: Once testing is complete and verified, I will update this PR with the necessary changes. PS: I'll be testing on a real device since it will be essential to fully validating the behavior of kUTTypeFileURL.

Update:
I tested this PR on a physical device and every file still goes through kUTTypeURL, I'm not proficient in Swift but everything works fine :)

@kayode0x kayode0x marked this pull request as ready for review June 3, 2024 02:16
@MaxAst
Copy link
Owner

MaxAst commented Jun 4, 2024

awesome, thank you @kayode0x! will test it later today

@kayode0x kayode0x changed the title [WIP] feat: implementing sharing files feat: implementing sharing files Jun 4, 2024
Copy link
Owner

@MaxAst MaxAst left a comment

Choose a reason for hiding this comment

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

LGTM! let's just remove the redundant text in the README and fix the package name

examples/with-file/README.md Outdated Show resolved Hide resolved
examples/with-file/package.json Outdated Show resolved Hide resolved
@kayode0x
Copy link
Contributor Author

kayode0x commented Jun 4, 2024

done :)

@MaxAst MaxAst merged commit d43802f into MaxAst:main Jun 5, 2024
@MaxAst
Copy link
Owner

MaxAst commented Jun 5, 2024

Released as part of 1.8.0: https://github.com/MaxAst/expo-share-extension/releases/tag/v1.8.0

Thanks for the contribution @kayode0x 🙏

@kayode0x
Copy link
Contributor Author

kayode0x commented Jun 5, 2024

happy to help!

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