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

Implement HandleEpisodeAction #18

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

Conversation

TheBlusky
Copy link
Contributor

I figured out GET /api/2/episodes/(username).json was not implemented.

I had to add relvacode/iso8601 dependencie in order to parse the since argument (time.RFC3339 is not compatible)

Hope I did everything the way you would have done it :-)

@TheBlusky
Copy link
Contributor Author

After using this branch on dalily basis, there is a bug with antenna pud trying to retrieve episodes.

(It sends since=0 which isn't an iso8601 date format. I'll work on that. In the mean time, don't merge it yet :-)

@oxtyped
Copy link
Owner

oxtyped commented Dec 26, 2023

Thanks for working on this! 🙇

@gilcu3
Copy link

gilcu3 commented May 10, 2024

@TheBlusky is there any progress on this? As far I can see the only thing missing is a corner case in the timestamp parsing? I can help if necessary.

@oxtyped Can devices currently synchronize episode actions without this PR? Sorry if it is a silly question.

@mhrivnak
Copy link

@TheBlusky any interest in finishing this? I may be interested in helping if that would be useful.

The since argument is odd. The API docs define it as an int, even though the POST API defines timestamp as A UTC timestamp ... in ISO 8601 format

So by a strict reading of the API, I think antennapod is correct in sending a 0, even though that seems weird.

@TheBlusky
Copy link
Contributor Author

@mhrivnak Sorry, I ended up stopping using gpodder2go. I won't finish this PR. It's more than a year old, maybe it has been implemented in main branch since ?

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.

4 participants