Hi Bhavani,<div><br></div><div>First of all, thanks a lot for the review.</div><div><br></div><div><div class="gmail_quote">On Fri, Jul 27, 2012 at 7:17 PM, Bhavani Shankar R <span dir="ltr"><<a href="mailto:bhavi@ubuntu.com" target="_blank">bhavi@ubuntu.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, Jul 23, 2012 at 11:56 PM, John Vrbanac <<a href="mailto:john.vrbanac@linux.com">john.vrbanac@linux.com</a>> wrote:<br>


> I have reviewed this app. The only change I needed to make was collapsing<br>
> the changelog down to one entry.<br>
><br>
> The code with my change is located here:<br>
> <a href="https://code.launchpad.net/~john.vrbanac/ubuntu-app-reviews/flashbox" target="_blank">https://code.launchpad.net/~john.vrbanac/ubuntu-app-reviews/flashbox</a><br>
<br>
</div>Dear John,<br>
<br>
Thanks a lot for your work.<br>
<br>
- found that licensecheck says that some files are in public domain<br>
<br>
bhavani@bhavani-spagetti-monster:~/arb$ licensecheck -r --copyright flashbox<br>
flashbox/setup.py: GPL (v3,)<br>
  [Copyright: 2012 Javed Khan <<a href="mailto:tuxcanfly@gmail.com">tuxcanfly@gmail.com</a>>]<br>
<br>
flashbox/flashcache.py: *No copyright* UNKNOWN<br>
<br>
flashbox/flashbox_lib/PreferencesDialog.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox_lib/Builder.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox_lib/helpers.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox_lib/flashboxconfig.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox_lib/AboutDialog.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox_lib/__init__.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox_lib/Window.py: *No copyright* Public domain<br>
<br>
flashbox/tests/test_lint.py: *No copyright* Public domain<br>
<br>
flashbox/tests/test_example.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox/FlashboxWindow.py: GPL (v3,)<br>
  [Copyright: 2012 Javed Khan <<a href="mailto:tuxcanfly@gmail.com">tuxcanfly@gmail.com</a>>]<br>
<br>
flashbox/flashbox/PreferencesFlashboxDialog.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox/AboutFlashboxDialog.py: *No copyright* Public domain<br>
<br>
flashbox/flashbox/__init__.py: *No copyright* Public domain<br>
<br>
whereas d/copyright says that files are released under gpl-3:<br>
<br>
Files: *<br>
Copyright: (C) 2012 Javed Khan <<a href="mailto:tuxcanfly@gmail.com">tuxcanfly@gmail.com</a>><br>
License: GPL-3<br>
<br></blockquote><div><br></div><div>Oops, I thought Quickly had taken care of that. I'll update the files missing copyright.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Also found the short description was equal to the long description in<br>
d/control which can be reworded better to be more descriptive I guess<br>
based on functionality of the app<br></blockquote><div><br></div><div>Yes, I'll add the long description.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<br>
Stopped the review here.<br>
<div class="im"><br>
><br>
> I installed the app and it appeared to function with vimeo; however, I<br>
> didn't test it past that.<br>
<br>
</div>Installed the app as a part of app showdown and found it just pops up<br>
a window with status as ready and a link to a video on vimeo and I<br>
couldnt get to know how the app functions. Would like a simple<br>
help/demo integrated into the app to have an overview of the app<br>
functionality I think personally.<br></blockquote><div><br></div><div>Are you using Google Chrome by any chance? The latest versions of chrome use PepperFlash</div><div>and this app only works with Chromium/Firefox and others using NPAPI flash. I guess John</div>

<div>must be using one of the supported browsers so it worked for him.</div><div><br></div><div>Could you please try opening the example vimeo url with Chromium/Firefox and check if it works for you?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div class="im"><br>
> I believe this app is ready for a vote.<br>
<br>
</div>Abstaining for now. +0 here.<br>
<br>
Regards,<br>
<span class="HOEnZb"><font color="#888888">--<br>
Bhavani Shankar<br>
Ubuntu Developer       |  <a href="http://www.ubuntu.com" target="_blank">www.ubuntu.com</a><br>
<a href="https://launchpad.net/~bhavi" target="_blank">https://launchpad.net/~bhavi</a><br>
</font></span></blockquote></div><br></div>