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

[plugin.audio.tripler] 3.0.0 #4392

Open
wants to merge 1 commit into
base: matrix
Choose a base branch
from
Open

[plugin.audio.tripler] 3.0.0 #4392

wants to merge 1 commit into from

Conversation

molzy
Copy link

@molzy molzy commented Oct 2, 2023

Description

A Kodi plugin for Triple R: an independent community radio station in Melbourne, Australia.

Version 3.0.0 is a complete revamp of this plugin.

  • Rewritten with more flexible parser
  • More content added from website
  • Improved user interface and menus
  • Browsing programs by name and date now possible
  • Broadcast track playlists are available
  • Support for playing music through Bandcamp, YouTube and indigiTUBE
  • Support for searching for music content by title and artist
  • Subscriber-only giveaways can now be entered by signing in

Code location: https://github.com/molzy/plugin.audio.tripler/

Checklist:

  • My code follows the add-on rules and piracy stance of this project.
  • I have read the CONTRIBUTING document
  • Each add-on submission should be a single commit with using the following style: [plugin.video.foo] v1.0.0

Additional information :

  • Submitting your add-on to this specific branch makes it available to any Kodi version equal or higher than the branch name with the applicable Kodi dependencies limits.
  • add-on development wiki page.
  • Kodi pydocs provide information about the Python API
  • PEP8 codingstyle which is considered best practice but not mandatory.
  • This add-on repository has automated code guideline check which could help you improve your coding. You can find the results of these check at Codacy. You can create your own account as well to continuously monitor your python coding before submitting to repo.
  • Development questions can be asked in the add-on development section on the Kodi forum.
  • If you see no activity on your PR after a week (so at least one weekend has passed) then please go to the #kodi-dev freenode IRC channel to reach out to the team

@romanvm
Copy link

romanvm commented Nov 20, 2023

@basrieter is this addon OK?

@basrieter
Copy link
Contributor

basrieter commented Nov 20, 2023

@basrieter is this addon OK?

Let me check in about 30 mins

Copy link
Contributor

@basrieter basrieter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have some comments that I would like to see addressed. Nothing big.

I do have to day that a bit more code commenting (especially some classes/methods) would have made it a lot easier to understand what you are doing.

plugin.audio.tripler/resources/lib/scraper.py Show resolved Hide resolved
plugin.audio.tripler/resources/lib/scraper.py Outdated Show resolved Hide resolved
plugin.audio.tripler/resources/lib/scraper.py Outdated Show resolved Hide resolved
plugin.audio.tripler/resources/lib/scraper.py Outdated Show resolved Hide resolved
plugin.audio.tripler/resources/lib/scraper.py Outdated Show resolved Hide resolved
plugin.audio.tripler/resources/lib/scraper.py Outdated Show resolved Hide resolved
plugin.audio.tripler/resources/lib/scraper.py Outdated Show resolved Hide resolved
Copy link
Contributor

@basrieter basrieter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please have a look at my (minor) comments.

@basrieter
Copy link
Contributor

Any update @molzy ?

@molzy
Copy link
Author

molzy commented Jan 8, 2024

Hey @basrieter , have been away for a while but just made the minor changes, thanks for taking a look.

Sorry about the lack of clarity in the code, it's a little bit abstracted. The intent of the scraper.py file is to contain the website scraping logic for the Kodi addon, while being usable outside of the Kodi addon as a standalone Python program to produce json output. Left over from implementing this functionality was the weak attempt at Py2 compat and the small usage of stderr.

I've removed both of these as per your comments since they aren't necessary for the standalone functionality. Thanks for the nudge to clean up! Let me know if there's anything else you would like me to clarify or change.

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

Successfully merging this pull request may close these issues.

None yet

3 participants