Skip to content
This repository has been archived by the owner on Jul 12, 2020. It is now read-only.

Update 18-appendix.md #529

Closed
wants to merge 1 commit into from
Closed

Update 18-appendix.md #529

wants to merge 1 commit into from

Conversation

zmilan
Copy link

@zmilan zmilan commented Dec 24, 2013

Since 'topic' is made dynamic and there is no track of it, off will never delete anything. So fast solution is to send event name (that can contain separation 'like module.part.event') and than delete all topics for it.

Since 'topic' is made dynamic and there is no track of it, off will never delete anything. So fast solution is to send event name (that can contain separation 'like module.part.event') and than delete all topics for it.
addyosmani added a commit that referenced this pull request Mar 9, 2014
@addyosmani
Copy link
Owner

Thanks! In your PR, you use a custom namespace (Fuisz?) which I think may have been your own as we don't use it in the appendix. I have however landed a change using the Cranium namespace and referred back to this issue so folks know you contributed it. Cheers!

@addyosmani addyosmani closed this Mar 9, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants