#18 Use Peertube API to publishAt instead of local utilities

Merged
LecygneNoir merged 2 commits from Zykino/prismedia:publishAt into develop 5 years ago
Zykino commented 5 years ago

Look like this work: the shedule update of the video is set on the web interface.

I'm forcing an upload with the --publishAt option to be private and then at the DATE I'm changing it to be public. I'm not sure how the --private and --publishAt functions interact with each other. (So for now the --publishAt overrides --private.)

Look like this work: the shedule update of the video is set on the web interface. I'm forcing an upload with the `--publishAt` option to be `private` and then at the DATE I'm changing it to be `public`. I'm not sure how the `--private` and `--publishAt` functions interact with each other. (So for now the `--publishAt` overrides `--private`.)
Zykino commented 5 years ago
Poster

Also maybe a part of this may be put in common in the utils (I largely inspire the code from the youtube lib, both uses iso formatted dates).

Open to discussions on how to update this PR.

Also maybe a part of this may be put in common in the utils (I largely inspire the code from the youtube lib, both uses iso formatted dates). Open to discussions on how to update this PR.
Poster
Owner

Hello

Thank you for this work!

From what I seen, this should perfectly work. The hard part (I had on Youtube :D) was to find and use correct format and you did it well!

I have not found information in the API documentation that the object scheduleUpdate is an array, well done. have I missing something about this or did you try until you found the correct syntax?

Anyway, I'll test the branch and merge it if everything is ok (and it should). No automatic test for the moment sorry :-(

Thanks!

Hello Thank you for this work! From what I seen, this should perfectly work. The hard part (I had on Youtube :D) was to find and use correct format and you did it well! I have not found information in the API documentation that the object scheduleUpdate is an array, well done. have I missing something about this or did you try until you found the correct syntax? Anyway, I'll test the branch and merge it if everything is ok (and it should). No automatic test for the moment sorry :-( Thanks!
Zykino commented 5 years ago
Poster

I asked on the Peertube IRC and they sended me to the doc here. And then when I asked the format of "datetime" they told me: "why don't you watch out for the request done by the webui?" And so... that was way easier this way ^^

I asked on the Peertube IRC and they sended me to the [doc here](https://github.com/Chocobozzz/PeerTube/blob/develop/support/doc/api/videos.yaml#L74). And then when I asked the format of "datetime" they told me: "why don't you watch out for the request done by the webui?" And so... that was way easier this way ^^
Poster
Owner

Ahah indeed, code still the more efficient documentation...
I used the compiled doc for the API and was unable to found how or where this video object is detailed :joy;

Ahah indeed, code still the more efficient documentation... I used the [compiled](https://github.com/Chocobozzz/PeerTube/blob/develop/support/doc/api/html/index.html) doc for the API and was unable to found how or where this video object is detailed :joy;
Zykino commented 5 years ago
Poster

Yeah, they told me the generated HTML doc is not complete when I pointed them there.

Yeah, they told me the generated HTML doc is not complete when I pointed them there.
LecygneNoir closed this pull request 5 years ago
The pull request has been merged as db94d96035.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.