Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • G google-photos
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • MemriMemri
  • plugins
  • google-photos
  • Merge requests
  • !1

bug fix + formatting

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Aziz Berkay Yesilyurt requested to merge aziz/photos into master Mar 16, 2023
  • Overview 3
  • Commits 2
  • Pipelines 2
  • Changes 5

There was a bug on file queue, we inserted photo instead of a file.

I have some general comments that would be nice to consider:

  • I cleanup requirement.txt to only list the direct requirements, it makes dependency handling easier. Let me know what you think.
  • I've spent some time to clean up queing logic in twitter-v2 plugin. I think it is pretty modular and includes some validation through pydantic. That would have also prevented the bug above. We could iterate on it and merge to pymemri. It is located here https://gitlab.memri.io/memri/plugins/twitter-v2/-/blob/dev/twitter_v2/pod_update_queue.py
  • There are some oauth2 scopes, which are not used. I removed them, let me know if they are needed.
    'https://www.googleapis.com/auth/drive.metadata.readonly',
    'https://www.googleapis.com/auth/drive.readonly',
    'https://www.googleapis.com/auth/gmail.readonly',
  • What do you think about using InstalledAppFlow.from_client_secrets_file to load the credential? I think it looks cleaner.
  • I was handling Oauth2 like this, just sharing to get your opinion, also as an input to unify the logic around implementing importers: https://gitlab.memri.io/memri/plugins/gmail_oauth/-/blob/dev/gmail_oauth/plugin.py#L52
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: aziz/photos