Thanks for the feedback. I changed this now.

*addon - plugin.image.photoapp
*version - 1.0.2
*url  - https://github.com/PhotoAppPlugin/plugin.image.photoapp/tree/v1.0.2
*revision - tag v1.0.2
*branch - master
*xbmc version - isengard


> Am 28.11.2015 um 15:57 schrieb ronie <ro...@kodi.tv>:
> 
>> On 17-11-15 00:45, Claude Marksteiner wrote:
>> *addon - plugin.image.photoapp
>> *version - 1.0.1
>> *url -https://github.com/PhotoAppPlugin/plugin.image.photoapp/tree/v1.0.1
>> *revision - tag v1.0.1
>> *branch - master
>> *xbmc version - isengard
> 
> hey Claude,
> 
> thanx for submitting your addon to our repo!
> 
> i've reviewed your code and have two minor change requests:
> 1) there's no need to put 'import xbmcvfs' in a try/except,
> this module is hardcoded in kodi and can't be removed, it will always be 
> available.
> 
> 2) use 'addon = xbmcaddon.Addon()', there's no need to specify the id there,
> it will be auto-resolved by kodi.
> 
> cheers,
> ronie

------------------------------------------------------------------------------
_______________________________________________
Xbmc-addons mailing list
Xbmc-addons@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/xbmc-addons

Reply via email to