hi,
thx for submitting.
Couple of thing.
id="plugin.video.rts-video" says video twice. id="plugin.video.rts"
should be enough :)
you can remove xbmc.json and xbmc.addon from the addon.xml
we want x.y.z. version number for consistency. so version="1.0.0"
summary/description/disclaimer should at least be provided in english as
well.
it should be <language>fr</language> as the videos are in French
can you make the fanart 720 or 1080 resolution? Should at least be in a
16:9 aspect ratio.
can you use language files for the settings.xml? Example:
https://github.com/XBMC-Addons/script.artwork.downloader/blob/master/resources/settings.xml
the IDs reference to
https://github.com/XBMC-Addons/script.artwork.downloader/blob/master/resources/language/English/strings.po
https://github.com/XBMC-Addons/script.artwork.downloader/blob/master/resources/language/French/strings.po
If you could fix those that would be great.
regards,
Martijn Kaijser
XBMC Foundation
On 30 August 2014 17:22, Gendre Sébastien <[email protected]> wrote:
> Hi.
>
> A new plugin.
> Hope I don't make any mistake. ;)
>
> *addon - RTS Video
> *version - 1.0
> *url - https://github.com/Yttrium39/xbmc-rts.git
> *revision - 8e8b4e03f5
> *branch - master
> *tag - 1.0
> *xbmc version - Gotham
>
> Have a good day. _o/
>
> --
> Gendre Sébastien <[email protected]>
>
>
>
>
> ------------------------------------------------------------------------------
> Slashdot TV.
> Video for Nerds. Stuff that matters.
> http://tv.slashdot.org/
> _______________________________________________
> Xbmc-addons mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/xbmc-addons
>
------------------------------------------------------------------------------
Slashdot TV.
Video for Nerds. Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
Xbmc-addons mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/xbmc-addons
