Skip to content

make handler classes public at module level#31

Open
ewheeler wants to merge 1 commit intomasterfrom
feature/make-handler-classes-public
Open

make handler classes public at module level#31
ewheeler wants to merge 1 commit intomasterfrom
feature/make-handler-classes-public

Conversation

@ewheeler
Copy link
Copy Markdown
Contributor

as of rapidsms 0.15.0, handlers must be explicitly
enumerated in settings.RAPIDSMS_HANDLERS
and this makes it slightly less cumbersome to do so:

so instead of:

RAPIDSMS_HANDLERS = ( 
    'appointments.handlers.confirm.ConfirmHandler',                             
    'appointments.handlers.move.MoveHandler',                                   
    'appointments.handlers.new.NewHandler',                                     
    'appointments.handlers.quit.QuitHandler',                                   
    'appointments.handlers.status.StatusHandler'
)

this will work:

RAPIDSMS_HANDLERS = ( 
    'appointments.handlers.ConfirmHandler',                             
    'appointments.handlers.MoveHandler',                                   
    'appointments.handlers.NewHandler',                                     
    'appointments.handlers.QuitHandler',                                   
    'appointments.handlers.StatusHandler'
)

as of rapidsms 0.15.0, handlers must be
explicitly added to settings.RAPIDSMS_HANDLERS
and this makes it slightly less cumbersome
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.

1 participant