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

Fix TypeError: this.ros.ActionClient is not a function #340

Merged
merged 1 commit into from
Aug 22, 2019

Conversation

Rayman
Copy link
Contributor

@Rayman Rayman commented Aug 22, 2019

I'm using the ES6 import statement to load roslib which works fine for
most roslib modules, except TFClient. That module doesn't properly
import it's dependencies but relies on the properties on the ros object.

I rewrote it a little bit to make it work for ES6 imports. Should be
backwards compatible.

I'm using the ES6 import statement to load roslib which works fine for
most roslib modules, except `TFClient`. That module doesn't properly
import it's dependencies but relies on the properties on the ros object.

I rewrote it a little bit to make it work for ES6 imports. Should be
backwards compatible.
Copy link
Contributor

@mvollrath mvollrath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mvollrath mvollrath merged commit c10d9ec into RobotWebTools:develop Aug 22, 2019
@Rayman Rayman deleted the fix/type-error branch August 23, 2019 07:31
MatthijsBurgh pushed a commit that referenced this pull request Jun 20, 2021
I'm using the ES6 import statement to load roslib which works fine for
most roslib modules, except `TFClient`. That module doesn't properly
import it's dependencies but relies on the properties on the ros object.

I rewrote it a little bit to make it work for ES6 imports. Should be
backwards compatible.
k-aguete pushed a commit to k-aguete/roslibjs that referenced this pull request Oct 21, 2022
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

Successfully merging this pull request may close these issues.

2 participants