-
Notifications
You must be signed in to change notification settings - Fork 0
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
File share Logic for sharing Video as File attachment #528
Comments
Update the design document and test cases as part of this task |
Worked on this. Was able to capture the details/metadata of video selected to be shared. Remaining part is to upload it on server |
Why we are not completing task and moving on a next task before completing them? |
Worked on this. There was an error. Image sharing and video sharing delegate methods had a conflict. I found that as i worked on this video sharing task, our previous file transfer of images and documents got broken. Looked into it and found the bug. Previously the delegate method "didFinishPickingImage" was called when image was selected and our all logic of image sharing was there. When we implemented video sharing, previous delegate method was not called and it called new delegate method "didFinishPickingMediaWithInfo" both for sharing images and videos. On video sharing, i have completed work for uploading video. Video is now successfully uploaded on server. Remaining is implementing video downloading part. It is in progress |
Thanks for the update. |
Video download logic has been completed. There is some error in saving and writing video in app sandbox. Looking into it. |
Ok |
I have finally resolved the issue with 528. Video is now correctly downloaded, saved and played on UI. Will mark this as resolved after some more testing |
This is done |
Please attache screen shots here of your test? Have you also tested between iOS and Android? |
Attached screenshots on 529. tried testing with sojharo but he said video part is not yet merged. so will test with him once he has merged in evening |
This was asked two days ago. When I ask a question I expect a response in a timely manner. |
I am ready to test it. Just give me 15 minutes. So that I finish the work on pull request by asad. After this I would test it with her. |
Testing with sojharo. Videos are received from android to iOS . Also from iOS to iOS. But not received by android if sent from iOS. We both are debugging to figure out if issue is on android side or iOS side. |
It looks like file type issue to me. As video is also working between two androids. However, the video type of iOS is video/quicktime or mov, and android can't play this type of video. I think by installing quick time player app for android might fix this. |
Please use Whatapp as the model and we need to support what is is supporting |
After some testing, we have found that it is working on iOS to android too. The problem was file is too big and takes time to download. I was closing the app from android studio without waiting longer. The correction for this should be to use some indicator to let user know how much file is downloaded. I would open task for this on android side so that we can track if file attachment is completely downloaded or not yet. |
Thanks |
Here we will implement logic for sharing videos as file attachment
The text was updated successfully, but these errors were encountered: