Skip to content

CAMEL-6999 Component for dropbox integration #58

Closed
wants to merge 22 commits into from

3 participants

@hifly81
hifly81 commented Nov 22, 2013

I've already opened a jira for this new component
https://issues.apache.org/jira/browse/CAMEL-6999

@hifly81
hifly81 commented Nov 23, 2013

I've included a txt file in resources about Dropbox Api Terms and Conditions

@hekonsek
hekonsek commented Mar 7, 2014

Hi Giovanni,

Apparently DropBox SDK is distributed on some MIT-like [1] DropBox license. Due to that reason we cannot ship it with the Apache Camel. However we could include it as a part of the Camel Extra (2) project.

Would you like to donate your code to the Camel Extra project?

Cheers.

(1) http://search.maven.org/remotecontent?filepath=com/dropbox/core/dropbox-core-sdk/1.7.6/dropbox-core-sdk-1.7.6.pom
(2) https://code.google.com/a/apache-extras.org/p/camel-extra

@hifly81
hifly81 commented Mar 7, 2014

Sure! no problem let's go for camel-extra

@hekonsek
hekonsek commented Mar 7, 2014

Great. Thank you for your contribution! I'll update the Jira ticket [1].

[1] https://issues.apache.org/jira/browse/CAMEL-6999

@hifly81
hifly81 commented Mar 7, 2014

Should I create a login to camel-extra jira?

@hekonsek
hekonsek commented Mar 7, 2014

Actually it seems that we will merge your code to ASF Camel, as the MIT license is compatible with [1] the Apache 2.0 license. You might be interested in tracking our discussion [2].

BTW Would you mind creating Karaf feature for your component? We will highly appreciate it.

Thank you again for contributing to the community. Cheers!

[1] https://www.apache.org/legal/3party.html#criteriaandcategories
[2] http://camel.465427.n5.nabble.com/camel-dropbox-td5748012.html

@hifly81
hifly81 commented Mar 7, 2014

Hi,
I will join the discussion ASAP.
about Karaf I will work to create a feature next week

@hekonsek
hekonsek commented Mar 7, 2014

That would be great - it will save us some work. Thanks in advance!

@hifly81
hifly81 commented Mar 7, 2014

NP it's a pleasure for me. Let me know about merging

@hifly81
hifly81 commented Mar 7, 2014

I've tried to post on nablle but I get
"This post has NOT been accepted by the mailing list yet."

@hekonsek

Hi Giovanni,

Try to register [1] to the list before posting via Nabble.

Cheers.

[1] mailto:users-subscribe@camel.apache.org

@hifly81
hifly81 commented Mar 13, 2014

By default now the integration test requiring a dropbox account are disabled.

@hifly81
hifly81 commented Mar 13, 2014

last commit added a features.xml --> camel-feature

@hekonsek

Sorry for the delay, Giovanni. I've been snowed with some other stuff :) .

One more thing before we merge - would you be so kind and fix the CheckStyle errors in your code? You can see these after executing the build with the sourcecheck Maven profile:

mvn clean install -Psourcecheck

I can do it on your behalf if you want me to, but it will speed things up if you help me. I'd really appreciate that.

Cheers.

@hifly81
@hifly81
@hekonsek

Cool! I think I should find time tomorrow and proceed with the merge. Thank you Giovanni!

@hifly81
hifly81 commented Apr 14, 2014

Hi Henryk,
should I have access to confluence page to add the documentation?

Ciao,
Giovanni

@jfarrell
jfarrell commented Jun 8, 2014

Closing per INFRA-7876

@jfarrell jfarrell closed this Jun 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.