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.

Sign up for GitHub

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

Jump to bottom

Iptv matrix #239

Merged
merged 15 commits into from Sep 8, 2019
Merged

Iptv matrix #239

merged 15 commits into from Sep 8, 2019

Conversation

phunkyfish
Copy link
Member

@phunkyfish phunkyfish commented Aug 10, 2019 •
edited

v4.7.0

  • Added: Support for IPTV Streams configured on E2 device (no timeshifting)
  • Added: Reload instead of reconnecting when channel/group changes are detected
  • Added: Use truly unique IDs for channels so EPG changes are correctly reflected
  • Fixed: Only get drive space for devices that have an HDD
  • Fixed: use correct function to lookup group when adding
  • Added: update README.md to show appveyor/travis badges per branch
  • Added: Update OSX build script
  • Added: update badge status for travis/appveyor
  • Added: add copyright notices to files
  • Fixed: Fix default path for genre text mapping file

@phunkyfish phunkyfish force-pushed the iptv-matrix branch 2 times, most recently from 2d95ba7 to 6cb094b Compare August 11, 2019 08:22
@phunkyfish phunkyfish requested a review from ksooo August 11, 2019 08:38
@phunkyfish phunkyfish force-pushed the iptv-matrix branch 3 times, most recently from bf36b4a to 319f268 Compare August 18, 2019 20:30
@phunkyfish phunkyfish force-pushed the iptv-matrix branch 16 times, most recently from 0de3636 to f990fe0 Compare August 27, 2019 21:01
This was referenced Aug 28, 2019
Closed
Closed
@phunkyfish
Copy link
Member Author

@ksooo this is ready for review when you have the time.

@phunkyfish phunkyfish force-pushed the iptv-matrix branch 2 times, most recently from 8c7b57f to 3875899 Compare August 28, 2019 21:06
@AlwinEsch
Copy link
Contributor

That's a good and great rework. Hopefully, PVR will soon be switched to the C ++ interface.

If you have some ideas for rework. add it here 😄: xbmc/xbmc#16485
Maybe you can already prepare for the addon in part (if possible).

@phunkyfish
Copy link
Member Author

Ty, yes, that’s the plan for the next PRs for vuplus and iptvsimple.

ksooo
ksooo reviewed Sep 6, 2019
View reviewed changes
Copy link
Member

@ksooo ksooo left a comment

Choose a reason for hiding this comment

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

some remarks/questions

src/Enigma2.cpp Outdated Show resolved Hide resolved
src/client.cpp Outdated Show resolved Hide resolved
src/client.cpp Outdated Show resolved Hide resolved
src/enigma2/data/Channel.cpp Outdated Show resolved Hide resolved
src/enigma2/Channels.cpp Outdated Show resolved Hide resolved
src/enigma2/Channels.cpp Outdated Show resolved Hide resolved
src/enigma2/Channels.h Outdated Show resolved Hide resolved
pvr.vuplus/addon.xml.in Outdated Show resolved Hide resolved
@phunkyfish
Copy link
Member Author

Ok, should be ready. Same changes made to Leia PR.

@phunkyfish
Copy link
Member Author

Thanks!

@phunkyfish phunkyfish merged commit d5c9a74 into kodi-pvr:Matrix Sep 8, 2019
@phunkyfish phunkyfish deleted the iptv-matrix branch September 8, 2019 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ksooo ksooo ksooo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@phunkyfish @AlwinEsch @ksooo