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

Polymer 1 support #2

Open
mvollrath opened this issue Jan 10, 2019 · 3 comments
Open

Polymer 1 support #2

mvollrath opened this issue Jan 10, 2019 · 3 comments
Assignees

Comments

@mvollrath
Copy link

mvollrath commented Jan 10, 2019

I'm working on a Polymer 1 project that uses ros-websocket. We need to use the newest (not yet released) version of roslibjs which adds CBOR binary encoding support.

For development, I've forked this repo at v3.0.2 with the new roslib version. We'd like to use a tagged release instead.

Since roslib doesn't have a major version yet, it seems like there would need to be a branch for 3.x and release every time roslib releases to keep it up to date and prevent breaking changes.

I'm appealing to roslibjs for a major version release so we can reduce the maintenance overhead of 3.x.

@chapulina
Copy link

@jstnhuang, let us know if you're interested in keeping a branch that supports Polymer 1. Otherwise, we will fork this under the osrf org and use that for RVizWeb instead (at least while we're still using Polymer 1). Thanks!

@jstnhuang
Copy link
Owner

Hi @mvollrath and @chapulina,

Thanks for keeping me updated about these issues. I'll just say that I made the 4.0 version of ros-websocket mostly as an experiment to see how hard it would be to move things to Polymer 3. I'd be happy to have one branch for Polymer 1/2 and another for Polymer 3. I think that if I were in your position, I would just want to fork this repo. That would give you the most control over the code, so you can make changes whenever you want.

That being said, I appreciate being updated on the changes you're working on. I'll definitely try updating roslib to v1.0.0 and creating a new branch sometime.

@jstnhuang jstnhuang self-assigned this Jan 25, 2019
@chapulina
Copy link

Thanks for the feedback!

@mvollrath , I created forks https://github.com/osrf/polymer-ros-websocket and https://github.com/osrf/polymer-ros-service, feel free to make PRs to those repos and change other components to use them for now. Thanks!

jubeira pushed a commit to jubeira/ros-websocket that referenced this issue Mar 11, 2019
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

3 participants