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.

Sign up for GitHub

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

Jump to bottom

Rate limit #55

Merged
merged 8 commits into from
May 26, 2017
Merged

Rate limit #55

merged 8 commits into from
May 26, 2017

Conversation

tnguyen14
Copy link
Owner

@tnguyen14 tnguyen14 commented May 18, 2017 •
edited
Loading

This will fix #45

  • Update drive API tests

@tnguyen14
pm2 logs will show all services
3ef6f8b
@tnguyen14
only retrieve metadata on GET /
6ad73de 
The initial GET `/` call will now result in 2 calls to Google Drive API,
instead of n calls, with n corresponding to the number of notes, as
before.
@tnguyen14
add endpoint to retrieve file content
4308c24
@tnguyen14
only get note content upon selecting a note
49c3940
@tnguyen14
render local version of note first
5ea1114
@tnguyen14
pin navigo at v4.6.0 due to breaking change
10928a9 
see krasimir/navigo#129
@tnguyen14
use md5Checksum to reconcile diff on first load
4397057 
if there's difference between local and remote, get remote content to
store, either to old note, or note.
@tnguyen14
update drive api tests
0fd0088
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.8%) to 66.923% when pulling 0fd0088 on rate-limit into e950e7f on master.

@tnguyen14 tnguyen14 merged commit 0fc2b9a into master May 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Drive API rate limit exceeded
2 participants
@tnguyen14 @coveralls