-
Notifications
You must be signed in to change notification settings - Fork 68
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
[DRAFT] Add Google Drive Integration #241
base: master
Are you sure you want to change the base?
Conversation
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.
CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
ece6064
to
cba4107
Compare
Even using the official Google Drive integration, you can't access JPGs or PNGs, so I don't think it should be a focus |
Probably google has some documentation, and then just point the user to where to input some tokens. |
I investigated this and I found that I can't get the icon pictured to show up in 2.15.1.1189 (latest toltec) or 3.5.2.1807 (latest stable). Neither with a shared folder to a specific account, or a shared via a link anyone can access. The google drive web page shows the "shared folder" icon so I think that would be a scenario where it would be shown in the remarkable as well. I looked at the network traffic as well and I didn't see anything that would indicate changing a folder icon. For these reasons I don't think its worth while to pursue this icon feature anymore. It's possibly discontinued by remarkable and the field is no longer sent to the clients. The older versioned clients may still respect it, but there is no way to tell anymore. |
image/png base64 encoded. resolution: 156x220 Example: Request:
Response:
Open this link in your browser to view the image:
|
d019821
to
4c5b597
Compare
Hi @NotNotQuinn! Thanks for your investigations! I just update my PR to include thumbnail retrieval, it works well for PDF, but not for google documents. Is this the same with the official integration? I use this patch on my own instance since June. Once a week (exactly 1 week after the integration creation), the token seems to expire unexpectedly. Initially the |
This isn't the case. On the official integration you can see previews for pdfs, google docs, google sheets, and google slides. Also, I wanted to mention:
This doesn't show up in the official google drive integration either. It feels like it should, and it would be cool if it did, but it doesn't. |
Update messages.IntegrationMetadata struct as mention in ddvk#241 (comment)
Update messages.IntegrationMetadata struct as mention in ddvk#241 (comment)
fb4ba9f
to
338115d
Compare
338115d
to
a4a0bb5
Compare
a4a0bb5
to
fe59e33
Compare
This PR implements Google Drive integration (it respond partially to #102). It depends on #240, and extends it a bit.
It features:
To finish my work, I need to find the answer to the following questions:
There is a dedicated icon to highlight shared drives (penultimate image on https://support.remarkable.com/s/article/Integrating-with-Google-Drive-Dropbox-and-OneDrive): what is the key missing for that inapparently notmessage.IntegrationFile
? (seems to be added in 2.13)message.IntegrationFile
?To do:
Understand why the doc told us JPG, PNG are supported, but they are reported as "Not compatible" with their official MIME type. Are they converted? How?seems not compatible after allLet my know if you have some hints, requests, ...