search

daveagp / websheets

Online version of coding handouts with blanks
GNU Affero General Public License v3.0
9 stars 8 forks source link

notify_new is not triggered for "copied" sheets #5

Closed dz0 closed 9 years ago

dz0 commented 9 years ago

I noticed this as my server doesn't have smtp configured and throws error on trying to mail.

So the error is thrown on creation of new (blank) sheet, but not on copying sheet to new one.

daveagp commented 9 years ago

Thanks for noticing both of these issues. I'll do my best to fix them tomorrow since I just got back from a trip.

dz0 commented 9 years ago

No rush :-) On Sep 22, 2015 7:28 AM, "Dave Pritchard" notifications@github.com wrote:

Thanks for noticing both of these issues. I'll do my best to fix them tomorrow since I just got back from a trip.

— Reply to this email directly or view it on GitHub https://github.com/daveagp/websheets/issues/5#issuecomment-142179389.

daveagp commented 9 years ago

Should be resolved by https://github.com/daveagp/websheets/commit/6fb09d3ef47b3f09ad7d86f6531c6a7d2d9d4c14