-
Notifications
You must be signed in to change notification settings - Fork 285
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
Sign in with Google temporarily disabled for this app #82
Comments
Yep, correct. This is a duplicate of </issues/70> which was closed in favor of </issues/64>. My fix is almost exactly what you suggest but cleaner as it avoids that "ugly workaround" of hacking the code as you suggested. The PR is open for your review at </pull/74>. (Your fix is what I've been suggesting to everyone who needs this working ASAP while the PR is still open.) |
Thanks for this amazing project! It was super useful to me. |
[UPDATE: The permanent fix described below has been merged BUT, the release tag hasn't been updated, so you'll need to compile and replace the old one in I provided a temporary solution for those who needs this working ASAP and filed a PR with a permanent fix, but that's undergoing review at this time. |
When using the app for the first time and going through the authorization steps, google says:
When authenticating, Google complained they didn't check the app and hence it's disabled.
I fixed this by going to the Google Cloud Console, I guess you have to be a developer to have this.
Then I created a project 'md2googleslides' and I created a 'OAuth Client ID' for 'Desktop app' also with the name 'md2googleslides', this gave me a client ID and client Secret.
The ID and the secret I manually injected into
bin/md2gslides
, in particular/home/herbert/.node_modules_global/bin/md2gslides
for my installation. Around line 130, in theauthorizeUser()
function, there areclientId
andclientSecret
entries.I think this should work, but I'm still working some stuff out and it is an ugly work around.
The text was updated successfully, but these errors were encountered: