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 for #670 #671

Closed
wants to merge 1 commit into from
Closed

Fix for #670 #671

wants to merge 1 commit into from

Conversation

prjoh
Copy link

@prjoh prjoh commented Oct 26, 2021

Fix for #670

Public API Changes

None

Description

Fixes apparent typing error: List[List[callable]] -> List[callable]

#670

Fix for #670
@prjoh
Copy link
Author

prjoh commented Oct 26, 2021

This error was introduced by 8dea92f. The commit shows no indication for using a nested list in this case. This seems like a typical typo to me. Please fix this, as this is a long standing issue with the core library of rosbridge!

@eborghi10
Copy link

Hey guys, we're also experiencing issues and this fix will help us a lot. We can provide a failure in case you need to replicate it.

CC @DiegoHollmann

@jtbandes jtbandes mentioned this pull request Dec 2, 2021
@jtbandes jtbandes self-assigned this Dec 2, 2021
@jtbandes jtbandes closed this in #687 Dec 2, 2021
jtbandes added a commit that referenced this pull request Dec 2, 2021
**Public API Changes**
None


**Description**
Fixes #670 so multiple subscribers to the same topic no longer throws an error. Closes #671.
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.

3 participants