-
Notifications
You must be signed in to change notification settings - Fork 0
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
Harvest Chrome Extension - V1 #1
base: main
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.
@mayank2424 @ankesh7 I think this file can be removed and added to .gitignore
after removal
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.
@mayank2424 This file is following multiple indentation formatting. Could you please set it to 2 spaces and re-indent/prettify this file. I would suggest adding a .prettierrc.js
file if it helps. You can copy it from an existing gist online.
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.
@prateekkathal I'll address this change. Thank you!
createStartTimerButton("google-calendar", eventHeaderElement, harvestTimerID, eventId, eventName); | ||
|
||
window._harvestPlatformConfig = { | ||
"applicationName": "Crowdlinker", |
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.
@mayank2424 @ankesh7 Can we move these to a common config.js
so it's easier for other open source users to modify for their own needs?
Same for any RGB/Hex colors that you feel will be nice to haves for the community.
Changes
Extension Preview