ryanjm / rails-event-manager

Repeating schedule manager Rails.
MIT License
0 stars 0 forks source link

Clean up next_occurrence #11

Open ryanjm opened 10 years ago

ryanjm commented 10 years ago

next_occurrence is one of the longest methods in the gem right now. It's logic should be broken up / cleaned up.

This might require rethinking how some of the methods are plugged together. One of the things I recently figured out / added was the ability to handle timezones. Right now that kind of feels like an add-on. That should be taken into a little more consideration during the refactoring process.