Skip to content
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

Gallery Added for Issue#6 #17

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Gallery Added for Issue#6 #17

wants to merge 3 commits into from

Conversation

nikhil3000
Copy link
Contributor

There are no more merge issues.

Copy link
Member

@anistark anistark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make these changes and push again. Also, close the previous PR. This is creating duplicate PR for same code.

server/routes/index.js Show resolved Hide resolved
token.json Outdated
@@ -0,0 +1 @@
{"access_token":"ya29.Gls6BudTLH6IErPdUS1xPQMPUOPwvJVwneAYO4j0PIG4b-yewUeOJllQszAs_QuhKB_VT4TeSGQhke6boSA8Zq_pdMUP6Qgh-4xaC6ix5W4qxNfYlj-Ob4OI1zm-","refresh_token":"1/2efG7YHnZLdrN_Oy3NeaPKMU1aEPWezij9e4n4yo0g4","scope":"https://www.googleapis.com/auth/drive.metadata.readonly","token_type":"Bearer","expiry_date":1539853462716}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't add tokens directly as commit. Take it from environment or in this case, config.js which won't be committed.

Remove token.json file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

token.json will be created when you'll use your credentials for google drive for the first time, after you'lll enter a one time access code generated by going to a url.

adding it to .gitignore so that it won't be commited.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not mandatory to create token.json
You can place the token in other places or on client side as well. Figure out a way to NOT use token.json

Also, adding to .gitignore won't delete this just yet. You've already committed this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll try to not use token.json, for the time being, I have deleted it before my latest push so it's no longer present in the latest version.

@@ -25,5 +25,17 @@ module.exports = {
},
newRelic: {
licenseKey: process.env.newRelicLicenseKey || ''
},
installed: {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

name it as googledrive. installed doesn't make sense.

@@ -25,5 +25,17 @@ module.exports = {
},
newRelic: {
licenseKey: process.env.newRelicLicenseKey || ''
},
installed: {
client_id: "397600756791-ekv992pp3cugmplk8ttk7g1al3up88lk.apps.googleusercontent.com",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove all the keys from config_example.js. Only write keys here and values as blank. The values can be added in the config.js file.

@nikhil3000
Copy link
Contributor Author

drive token is now being saved to config.json file

@anistark
Copy link
Member

If token.json is removed, why is it still in the code?

@nikhil3000
Copy link
Contributor Author

Where?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants