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

Update all YT claims with new metadata #18

Closed
tzarebczan opened this issue Jan 28, 2019 · 5 comments
Closed

Update all YT claims with new metadata #18

tzarebczan opened this issue Jan 28, 2019 · 5 comments
Assignees
Labels
priority: blocker Issue is blocking release, do ASAP sdk dependent Tom's Wishlist type: new feature New functionality that does not exist yet

Comments

@tzarebczan
Copy link
Contributor

This will serve as the ytsyc issue to implement the below and an epic to track types/sdk required changes. When the sdk tickets are filed, I'll update it.

  1. Setup a process to perform claim updates, preferably in batch mode
    (Allow batching of transactions, including publishes/updates lbry-sdk#1821). If easy update mode is available (Claim updates via publish require superfluous paramaters lbry-sdk#1423), use it, if not, read from existing claim data. This also includes grabbing the sdblob for each claim in order to populate some of the new file related fields.

  2. From the sdblob, grab the filename and file size to populate name/size in Add file attributes to metadata types#9.

  3. Grab video length from the YouTube api to populate length in Add file attributes to metadata types#9

  4. Grab published date from YouTube api to populate releaseTime in Add releaseTime to schema types#13

  5. Grab category/tags from YouTube api to populate in Add category to schema types#15 or Add tags to schema types#16

  6. Grab language from YouTube api and populate language on claim metadata

  7. Research other relevant youtube API data that may require new types/sdk entries.

@tzarebczan tzarebczan added Tom's Wishlist consider soon Discuss this issue at the next planning meeting, then remove this label priority: medium Work needs to be done within 2-3 sprints type: new feature New functionality that does not exist yet sdk dependent labels Jan 28, 2019
@alyssaoc alyssaoc added priority: blocker Issue is blocking release, do ASAP and removed consider soon Discuss this issue at the next planning meeting, then remove this label priority: medium Work needs to be done within 2-3 sprints labels Feb 6, 2019
@alyssaoc
Copy link

What is the status of this as a blocker

@kauffj
Copy link
Member

kauffj commented Mar 25, 2019

@nikooo777

  1. Can this ticket be updated to include adding the relevant creator metadata as well?
  2. Can you please tag @lbryio/appsquad as soon as the first new metadata exists in the blockchain?

@tzarebczan
Copy link
Contributor Author

Niko and I ran through this today and all the fields we need are available in the YouTube api. The email we'll populate from their LBRY account. We'd have to link back their featured content URL to a published file which is the trickiest part since we'll need to know the URL beforehand or update it later.

I'll get a full sample of a channel publish to the app squad soon.

@kauffj
Copy link
Member

kauffj commented May 10, 2019

@nikooo777 this just came up in a discussion with @robvsmith: we should pull in the tags already set in internal-apis and apply those as well

@lyoshenka lyoshenka removed this from the Mar 27th (API) milestone May 15, 2019
@tzarebczan
Copy link
Contributor Author

This is completed, please reopen if that's not the case @nikooo777

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: blocker Issue is blocking release, do ASAP sdk dependent Tom's Wishlist type: new feature New functionality that does not exist yet
Projects
None yet
Development

No branches or pull requests

5 participants