Joystream / youtube-synch

YouTube Synchronization
11 stars 10 forks source link

Issues with synced videos #81

Closed drillprop closed 1 year ago

drillprop commented 1 year ago

Questions:

dmtrjsg commented 1 year ago

@drillprop thank you for putting this together, super hepful! While @zeeshanakram3 is best to comment on the problems you outlined i'll try to chip in on the questions part:

publishedBeforeJoystream - is this already created on QN schema with the purpose of capturing exactly this use case or are you suggesting to add it? So recency can and possibly will be used in the recommendation and search relevancy rules so having some sort of indication of original date is helpful indeed ✅

Do you know what was hasMarketing designed for ❓

On Propertie:

  1. What do we do with language? - there isn't any language property associated with video that we can query, YT channels do have defaultLanguage , we can propagate that to all synced videos
  2. What do we do with Explicit?
    • if not possible to fetch from YT, then leave this flag as false by default, and content moderation team can do their work. I don't think this will be a huge problem for us given the content focus of Gleev on Web3 creators.
  3. What do we do with License?
    • If not possible to fetch from YT meta, then we need to assign default CCO as it is in atlas and add a note to the OB flow that for any videos with limited license this would need to be updated manually.
dmtrjsg commented 1 year ago

Resolution: publishedBeforeJoystream (date field) will be left empty. One possible use case is to use it in the recommendation and search relevancy rules so having some sort of indication of original date is helpful ONLY if we are pulling in historical videos.