-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
feat: add Google Photo support #1853
Conversation
Thanks for opening this pull request! Please check out our contributing guidelines. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果Google Photo有目录的概念的话,可能需要在添加按照目录列出后merge
} | ||
|
||
func (d *GooglePhoto) List(ctx context.Context, dir model.Obj, args model.ListArgs) ([]model.Obj, error) { | ||
files, err := d.getFiles() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里没有使用file_id 是一次性展示所有吗
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
根据谷歌提供的api来说,除了列出所有,或者通过手动进行搜索条件筛选列出、列出影集这几种。
我这边也review了一下rclone的做法,他们那边好像也是遍历分日期、影集、全部这三种方式展示全部文件(包括重复的情况)。
var config = driver.Config{ | ||
Name: "GooglePhoto", | ||
OnlyProxy: true, | ||
DefaultRoot: "root", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果Google photo没有排序的方法的话,可以在这里加上LocalSort:true
以使用本地排序。
谷歌相册好像目前只有影集这种目录的情况,但是需要手动在相册app添加相片至影集才行。 |
Congrats on merging your first pull request! We here at behavior bot are proud of you! |
增加对谷歌相册的支持。
谷歌相册api相当的多限制,只能支持下载和上传,其他操作基本上没了。
还有几个问题:
可以改进的地方: