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

Add missing fields for testing #155

Closed
wants to merge 1 commit into from
Closed

Conversation

emveepee
Copy link
Contributor

To assist testing without a backend, add SetYear() to Recordings and EPG and add SetEpisodePartNumber() to EPG.

Note currently EpisodePartNumber is not exposed for skinnning.

Add SetEpisodePartNumber() to EPG, add SetYear() to Recordings and EPG.
emveepee added a commit to emveepee/pvr.demo that referenced this pull request Jul 28, 2024
A few available tags where not implemented for testing.  Update program and test file.  Port of kodi-pvr#155.
emveepee added a commit to emveepee/pvr.demo that referenced this pull request Aug 20, 2024
A few available tags where not implemented for testing.  Update program and test file.  Port of kodi-pvr#155.
@ksooo
Copy link
Member

ksooo commented Aug 20, 2024

@emveepee closing because this cannot go into Omega.

@ksooo ksooo closed this Aug 20, 2024
@emveepee
Copy link
Contributor Author

emveepee commented Aug 20, 2024

@emveepee closing because this cannot go into Omega.

Thanks on PR 159 I noted this should be canceled I wasn't sure how. BTW there is nothing wrong with the PR in Omega, just no way to display it.

@ksooo
Copy link
Member

ksooo commented Aug 20, 2024

Why should we merge code of no value into the omega addon? This makes absolutely no sense.

@emveepee
Copy link
Contributor Author

Why should we merge code of no value into the omega addon? This makes absolutely no sense.

Agreed which is why asked for it to taken out but it is not an absolute no. Not all tests have to be against a skin, initially I used it to check that the database was being updated correctly.

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

Successfully merging this pull request may close these issues.

2 participants