data-transfer-project icon indicating copy to clipboard operation
data-transfer-project copied to clipboard
verified publisher icon google

Photobucket integration

Open vasylshche opened this issue 4 years ago • 5 comments

This code is already in use by FB and PB. Contains:

  • photos importer
  • videos importer
  • auth extension

vasylshche avatar Dec 13 '21 14:12 vasylshche

This pull request introduces 2 alerts when merging 7e6453601c16447f55fe79e1c7c9e73d74973925 into 7d263e0d40bb28ef307d6242fb76a64ade686f2a - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak

lgtm-com[bot] avatar Dec 30 '21 22:12 lgtm-com[bot]

This pull request introduces 2 alerts when merging b8f8d9b1dbe6aa5fb880206782e95eef0ff2588c into 7d263e0d40bb28ef307d6242fb76a64ade686f2a - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak

lgtm-com[bot] avatar Dec 30 '21 23:12 lgtm-com[bot]

This looks good to me in general. Are there plans to also add an exporter? We typically shy away from importer only adapters in case someone is trying this project out locally or otherwise and wants to get their data exported from the third party service

seehamrun avatar Jan 25 '22 21:01 seehamrun

Are there plans to also add an exporter?

@seehamrun Most probably not in the nearest future, for several reasons. One of them - for now we are working on the new FE app and as a result rewriting out graphql read layer, so if we do any steps forward writing of the exporter - most probably we will have to significantly update it eventually.

vasylshche avatar Feb 01 '22 08:02 vasylshche