[Native Share] Implement Native Share for IOS

[grgia]

Description

This is a tracking issue for the IOS implementation of the Native Share project. This section of the project will be handled separately but merged alongside the Android issue.

Project

https://docs.google.com/document/d/14C1VifxvIXeyLAf0XstbC-wvyAb5SQZ0AOw0-jLRyZs/edit

Tracking

#36613

[BrtqKr]

Hey, I'm from SWM, I'd like to take care of this issue.

[melvin-bot]

@grgia, @BrtqKr Whoops! This issue is 2 days overdue. Let's get this updated quick!

[melvin-bot]

@grgia, @BrtqKr 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

[melvin-bot]

@grgia, @BrtqKr 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

[melvin-bot]

@grgia, @BrtqKr 10 days overdue. Is anyone even seeing these? Hello?

[grgia]

Bumped for update / steps for unblocking permissions

[BrtqKr]

I am focusing on split navigation atm. I'll try to check the setup whenever it's applied and try to continue whenever the navigation is ready.

[melvin-bot]

@grgia, @BrtqKr Whoops! This issue is 2 days overdue. Let's get this updated quick!

[BrtqKr]

Taking care of this atm. I've checked the groups before the weekend and they seemed to work as expected.

[BrtqKr]

Working on the files processing - they are being saved correctly from the share extenbsion, but there's something wrong with retrieving them on the react native side. This might be something related to the permissions, but I'll try to confirm it tomorrow.

[grgia]

Thanks @BrtqKr- do you have a draft PR up for your branch?

[BrtqKr]

Not yet, wired it up though and the files are being passed to the react native wihthout any issues, so happily there's no need for the additional set-ups. I'll be verifying the scanning/sending with the passed files tomorrow, so this would probably be the initial version for the ios. I'll share more details next evening.

[BrtqKr]

Moving forward with that - wired up most of the js parts - I'll continue working on that and the files cleanup after sharing tomorrow

[grgia]

Sounds great, thanks @BrtqKr

[BrtqKr]

Still working on the JS part - today it was mostly persistence and onyx-related things, which are necessary for removing files - it's not done yet though

[BrtqKr]

Continued the topic from yesterday. I've found some edge cases about the cleanup part - there's an issue when the share attempt gets cancelled and the file isn't removed properly, but we can handle this by performing cleanups for the files older than a week. Besides I had to slightly modify the model for the image, but apply small updates to the doc when I'm sure everything works.

[grgia]

Great @BrtqKr, do you have an idea for when we might be able to get this into the review stage?

[BrtqKr]

@grgia, I've finished wiring up the core version for the ios today. Also, @filip-solecki mentioned that there wasn't much left regarding the Android native part, so I'd say that the remaining things before the PR would be:

1. Adjust the data format on Android to match the ios (I had to slightly change the format to handle some cases on the react-native side)
2. Finish the file removal
3. Adjust the display for the files
4. Pass to @staszekscp and @war-in to apply the hybrid app-related changes and test it properly

The first three points shouldn't take much time since those are just the follow-ups to the already existing parts (the second point is a bit troublesome to test though). I'm not sure about the fourth point. As long as everything goes as expected on the hybrid part we should be able to send the final version sometime at the beginning the next week, but this is something @staszekscp or @war-in would need to estimate when they start applying the changes.

Simulator.Screen.Recording.-.iPhone.15.Pro.-.2024-10-08.at.02.19.43.mp4

[filip-solecki]

Hi! Quick update - I took the issue from @BrtqKr and I'll work on both right now as JS part will be shared for both Android and iOS

[melvin-bot]

@grgia, @BrtqKr, @filip-solecki Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[grgia]

@cdOut @filip-solecki Could you post a current update?

[filip-solecki]

Sure, @cdOut is fixing image displaying issues which happen sometimes and he also finished Submit flow so we have to finish Android and merge it here and we can push it to review and tests and adjust it later to HybridApp

[melvin-bot]

@grgia, @BrtqKr, @cdOut, @filip-solecki 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

[melvin-bot]

@grgia, @BrtqKr, @cdOut, @filip-solecki 10 days overdue. Is anyone even seeing these? Hello?

[grgia]

@filip-solecki cc @cdOut mind posting a quick update here as well?

[cdOut]

Hi, I'm finishing up the Submit flow and share media display issue while Filip worked on handling other filetypes and merging the IOS and Android branches of the Share flow.

[grgia]

Great thank you! @cdOut

[grgia]

@cdOut do you have an EOW update?

[cdOut]

@grgia I was just a side helper on this issue during Filip's OOO, if any updates on the state of it are in order you should ping @filip-solecki as he's more informed on the state of it and is the main person behind the feature. I am also no longer working for Software Mansion so I won't be able to help with that.

[filip-solecki]

Hi @grgia, so on the iOS side most of the issues have been resolved, now we need to block sharing for unsupported file formats and address some minor JS issues

[grgia]

Great @filip-solecki could you give me an update on when you think we could get a draft PR up?

[filip-solecki]

Hi! So I think that the realistic date may be Wednesday/Thursday in the next week

[grgia]

Great, keep me posted @filip-solecki

[melvin-bot]

@grgia, @BrtqKr, @cdOut, @filip-solecki Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[filip-solecki]

Hi @grgia ! Unfortunately I was sick most of this week and we won't be able to prepare PR today, it should be ready in the mid of next week I hope

[grgia]

@filip-solecki sorry to hear that, hope you're feeling better! How's this week for you?