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.

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

Any plans on updating for the new OrientJS? #124

Closed
vinerz opened this issue Aug 1, 2015 · 12 comments
Closed

Any plans on updating for the new OrientJS? #124

vinerz opened this issue Aug 1, 2015 · 12 comments

Comments

@vinerz
Copy link

vinerz commented Aug 1, 2015

Now that Oriento is deprecated, giving place to OrientJS maintained by OrientDB team, are there any plans to update this layer to the new library?

Thank you all for the wonderful work.

@nafetswirth
Copy link

+1

1 similar comment
@tommykennedy
Copy link
Contributor

+1

@dmarcelino
Copy link
Member

Eventually. Current npm version of OrientJS (v2.0.0) is orientechnologies/orientjs@e2f72c0 (16th June) which doesn't include PR codemix/oriento#330, so it's inferior to Oriento v1.2.0. After OrientJS releases a newer version with added features / fixed issues we'll move to OrientJS.

@nidaca
Copy link
Contributor

nidaca commented Aug 12, 2015

+1

1 similar comment
@AndreaPravato
Copy link

+1

@lvca
Copy link

lvca commented Sep 24, 2015

Hey guys, I merged it in OrientJS

@baltag
Copy link

baltag commented Sep 24, 2015

Thanks @lvca. Now @dmarcelino may start to integrate it into sails-orientdb.

@nidaca
Copy link
Contributor

nidaca commented Sep 24, 2015

👍 ... means this that we must prepare also for the 2.1 OrientDB snapshot in the same time with an update of the driver, or (for the start) it will work with the current 2.0 ?

@dmarcelino
Copy link
Member

That's cool @lvca but those changes haven't been released to npm:

screen shot 2015-09-25 at 09 57 44

@lvca
Copy link

lvca commented Sep 25, 2015

Hey @dmarcelino you're right. Sorry, but I'm not still very good at this Node.js stuff ;-) Done, right now.

dmarcelino added a commit that referenced this issue Sep 25, 2015
dmarcelino added a commit that referenced this issue Sep 25, 2015
@baltag
Copy link

baltag commented Sep 25, 2015

Great, thanks

@vinerz
Copy link
Author

vinerz commented Sep 25, 2015

Awesome! Thank you guys :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants