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

File share UI work for sharing video as file attachment #529

Closed
sumairasaeed opened this issue Feb 22, 2017 · 19 comments
Closed

File share UI work for sharing video as file attachment #529

sumairasaeed opened this issue Feb 22, 2017 · 19 comments

Comments

@sumairasaeed
Copy link
Contributor

Here we will do UI work needed for sharing Videos as file attachment

@sumairasaeed
Copy link
Contributor Author

Exploring and reading up on it how video can be shown and played on UI of iOS. Will update more here.

@sumairasaeed
Copy link
Contributor Author

Worked on this. Now it shows videos on UI when sharing files from where user can select the video to be shared.

@sumairasaeed
Copy link
Contributor Author

Completed work on this. Understood implementation video component for playing videos. Created a sample application with the component and was able to play a sample video. Also created that component in our application.

@jekram
Copy link
Contributor

jekram commented Feb 27, 2017

So should we mark it resolved? If the work is complete.

@jekram
Copy link
Contributor

jekram commented Feb 27, 2017

I am assuming that design document and test matrix have been kept current. Correct?

Please send me screen shot of the test

@sumairasaeed
Copy link
Contributor Author

Yes sir. design document is updated. Test matrix updated till last week. Will do more testing this week and update. Emailing you testing screenshots

@sumairasaeed
Copy link
Contributor Author

Tested after completing #528 . Video is correctly shown if one video is shared. If more than one video shared, then it shows black area and video is not played. Reopening and looking into it

@jekram
Copy link
Contributor

jekram commented Feb 27, 2017

Did we not tested multiple videos before?

@sumairasaeed
Copy link
Contributor Author

This has been fixed now. In this, we created and tested UI component for attaching and playing video inside player. Tested it by attaching video with an element.
Video plays as full screen when clicked. Same as in watsapp.

@jekram
Copy link
Contributor

jekram commented Feb 27, 2017

You did not answered my previous question?

Do you even read them or just chose to not answer them.

@sumairasaeed
Copy link
Contributor Author

Sorry. It was missed.
No sir. Was not able to test iOS to Android.
Here are screenshots for test results
screen shot 2017-02-27 at 3 50 50 pm
Uploading Screen Shot 2017-02-27 at 3.51.25 PM.png…
Uploading Screen Shot 2017-02-27 at 3.51.54 PM.png…
Uploading Screen Shot 2017-02-27 at 3.52.09 PM.png…
Uploading Screen Shot 2017-02-27 at 3.52.24 PM.png…
Uploading Screen Shot 2017-02-27 at 3.53.01 PM.png…
Uploading Screen Shot 2017-02-27 at 12.19.57 PM.png…
Uploading Screen Shot 2017-02-27 at 12.26.32 PM.png…
Uploading Screen Shot 2017-02-27 at 12.27.51 PM.png…
Uploading Screen Shot 2017-02-27 at 12.30.27 PM.png…
Uploading Screen Shot 2017-02-27 at 12.30.40 PM.png…

@jekram
Copy link
Contributor

jekram commented Feb 27, 2017

why we would not test with Android? when Android was getting developed there was no iOS so they could not test with iOS.

Would it not be logical to test with Android to make sure it is working? or you think we should do it later and reopen closed issues?

@sumairasaeed
Copy link
Contributor Author

Thats right that we should test with Android also. I have Android 4.4 on my phone and KiboChat app does not run on it. I would carry out tests by borrowing from sojharo or dayem tomorrow in office IA whoever is available.

@jekram
Copy link
Contributor

jekram commented Feb 28, 2017

There are multiple ways to test. They do not need to be in the office. You can slack or email them. I would be happy also to test.

@jekram
Copy link
Contributor

jekram commented Mar 1, 2017

What is the status here? My expectations is that when I put comment that I get an response in a timely manner.

@jekram jekram removed the Resolved label Mar 1, 2017
@sumairasaeed
Copy link
Contributor Author

Video part on android is not merged. sojharo will merge it in a while. will test then

@jekram
Copy link
Contributor

jekram commented Mar 1, 2017

Ok

@sumairasaeed
Copy link
Contributor Author

Tested. video sharing is working fine between android and iOS

@jekram
Copy link
Contributor

jekram commented Mar 1, 2017

Thanks for the update

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

No branches or pull requests

2 participants