search

praekeltfoundation / vumi

Messaging engine for the delivery of SMS, Star Menu and chat messages to diverse audiences in emerging markets and beyond.
BSD 3-Clause "New" or "Revised" License
420 stars 131 forks source link

Remove unused SimpleDispatcher #997

Closed jerith closed 8 years ago

jerith commented 8 years ago

It hasn't been touched in several years and it appears to be the only thing that uses publishers with arbitrary routing keys.

hodgestar commented 8 years ago

Looks good. There are also a reference to SimpleDispatcher in etc/supervisord.smpp_test.conf though.

hodgestar commented 8 years ago

Do we want to fix config/smpp_test.yaml and config/smpp_test_http_post.yaml so they have the same transport _name now that the dispatcher is gone?

jerith commented 8 years ago

No, because they're both transports. I think the stuff I removed was broken anyway.

hodgestar commented 8 years ago

Oh. :/

hodgestar commented 8 years ago

:+1: