search

adhearsion / voicemail

Adhearsion voicemail plugin
MIT License
8 stars 12 forks source link

Refactor Storage API #36

Closed wdrexler closed 9 years ago

wdrexler commented 10 years ago

Addresses #21 This will break backwards compatibility and should only be merged as part of a 2.0 release. Tagging @bklang, @JustinAiken, and @polysics for review

bklang commented 10 years ago

I'll merge this to develop shortly and officially begin the 2.0 development process.

Before I do, I'm going to make one last release of voicemail 1.

bklang commented 10 years ago

This needs documentation. Perhaps the easiest thing would be to add all the skeleton methods (to storage.rb?) and add RDoc to them. Then we can auto-generate the API docs.

bklang commented 10 years ago

Or, if not storage.rb, then a generic storage implementation class with empty methods that exists only for the purpose of documentation