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

Add box.com nodes #33

Merged
merged 3 commits into from
Oct 30, 2014
Merged

Add box.com nodes #33

merged 3 commits into from
Oct 30, 2014

Conversation

hindessm
Copy link
Contributor

@hindessm hindessm commented Oct 8, 2014

Some initial work on a box node. The code works but I need to:

  • add more logic to handle the behaviour when files already exist on the remote end,
  • make sure a keepalive-like token request occurs if no other requests have happened for a few days ensure the refresh token remains valid (it expires every 60 days otherwise)
  • fix msg.event to be more consistent with s3 and dropbox nodes.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 5353eed on hindessm:box-dot-com into a8b27a3 on node-red:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 6fee12b on hindessm:box-dot-com into 2d18c37 on node-red:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 23af00e on hindessm:box-dot-com into ec495e5 on node-red:master.

@hindessm hindessm force-pushed the box-dot-com branch 5 times, most recently from 105828c to 99f885f Compare October 13, 2014 08:54
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 99f885f on hindessm:box-dot-com into 8098f7b on node-red:master.

@hindessm hindessm force-pushed the box-dot-com branch 2 times, most recently from 8a6c3db to aee0500 Compare October 15, 2014 09:14
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling aee0500 on hindessm:box-dot-com into 8098f7b on node-red:master.

@hindessm hindessm changed the title [wip] Add box.com upload node. Add box.com upload node. Oct 15, 2014
@hindessm
Copy link
Contributor Author

This is ready for review/merge. Thanks.

@hindessm hindessm force-pushed the box-dot-com branch 2 times, most recently from 09ee0d9 to 7574aea Compare October 15, 2014 13:34
@hindessm hindessm changed the title Add box.com upload node. [wip] Add box.com upload node. Oct 21, 2014
@hindessm hindessm changed the title [wip] Add box.com upload node. [wip] Add box.com nodes Oct 21, 2014
@hindessm hindessm force-pushed the box-dot-com branch 2 times, most recently from 63e330e to 8e593ca Compare October 23, 2014 07:48
@hindessm hindessm changed the title [wip] Add box.com nodes Add box.com nodes Oct 23, 2014
knolleary added a commit that referenced this pull request Oct 30, 2014
@knolleary
Merge pull request #33 from hindessm/box-dot-com
55af7bd 
Add box.com nodes
@knolleary knolleary merged commit 55af7bd into node-red:master Oct 30, 2014
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.

None yet
3 participants
@hindessm @coveralls @knolleary