-
Notifications
You must be signed in to change notification settings - Fork 9
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
Extension build process #75
Conversation
@DrLynch Have you gotten a chance to review this process yet? We ought to get this merged in. |
"description": "Tracks writing in Google Docs, and provides nifty insights to you and your teachers!", | ||
"version": "0.0.1", | ||
"homepage": "http://chrome.google.com/webstore", | ||
"author": "ENTER YOUR NAME HERE", |
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.
Enter our names here.
"author": "ENTER YOUR NAME HERE", | ||
"repository": { | ||
"type": "git", | ||
"url": "ENTER GIT REPO URL" |
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.
Enter https://github.com/ETS-Next-Gen/writing_observer here (or even the path to the extension)
As per our conversation, I would be inclined to merge this sooner rather than later (e.g. when my two comments are addressed), and then keep working from there. We do also need to check for outstanding issues on the NCSU side, to avoid merge conflicts. Once the two changes are made, and we're clear with NCSU, I approve merge. |
@DrLynch we decided to merge this at it was a blocking step for Piotr. We can continue to work on beyond this. |
That's fine Brad I will try it out in any case and send comments when I get time. |
No description provided.