Parcourir la source

companion: clean up drive query

stephentuso il y a 5 ans
Parent
commit
fcb8d503be

+ 0 - 10
packages/@uppy/companion/src/server/provider/drive/adapter.js

@@ -56,16 +56,6 @@ exports.getItemId = (item) => {
 }
 
 exports.getItemRequestPath = (item) => {
-  // If it's from/is a Team Drive, add the Team Drive ID as a query param.
-  // The server needs the Team Drive ID to list files in a Team Drive folder.
-  if (exports.isTeamDrive(item)) {
-    return `${item.id}?teamDriveId=${item.id}`
-  }
-
-  if (item.teamDriveId) {
-    return item.id + `?teamDriveId=${item.teamDriveId}`
-  }
-
   return item.id
 }
 

+ 4 - 10
packages/@uppy/companion/src/server/provider/drive/index.js

@@ -24,11 +24,10 @@ class Drive {
   list (options, done) {
     const directory = options.directory || 'root'
     const query = options.query || {}
-    const teamDriveId = query.teamDriveId
 
     let teamDrivesPromise = Promise.resolve(undefined)
 
-    const shouldListTeamDrives = directory === 'root' && !teamDriveId && !query.nextPageToken
+    const shouldListTeamDrives = directory === 'root' && !query.nextPageToken
     if (shouldListTeamDrives) {
       teamDrivesPromise = new Promise((resolve) => {
         this.client
@@ -49,14 +48,9 @@ class Drive {
     let where = {
       fields: DRIVE_FILES_FIELDS,
       pageToken: query.nextPageToken,
-      q: `'${directory}' in parents and trashed=false`
-    }
-    if (teamDriveId) {
-      // Team Drives require several extra parameters in order to work.
-      where.supportsAllDrives = true
-      where.includeItemsFromAllDrives = true
-      where.driveId = teamDriveId
-      where.corpora = 'drive'
+      q: `'${directory}' in parents and trashed=false`,
+      includeItemsFromAllDrives: true,
+      supportsAllDrives: true
     }
 
     const filesPromise = new Promise((resolve, reject) => {