Skip to content

Conversation

@professor-k
Copy link

I've applied those changes, anything complicated. Also added localization for the last option you've added.

@professor-k
Copy link
Author

Crap. Sorry. It wasn't obvious that github will create new issue for the pull request. I'm new to github, and it actually took me more time to configure my account than to develop feature itself:(

@matchu
Copy link
Owner

matchu commented Apr 13, 2012

Oh, neat! Thanks for the help. I'm not going to have time to really look over this for probably a week or so (all sorts of deadlines are overlapping like crazy), but just wanted to let you know I saw this pull request and I really appreciate your help :D Thanks so much!

@matchu
Copy link
Owner

matchu commented Dec 6, 2012

Shoot, sorry I never really got back to you on this >_< This extension has been off my radar for a while.

I'm hoping to release a new version of the extension this month after exams. I'll probably end up merging the notification localization (thanks!), but I think I'll leave out auto mode for the time being as it seems like a really niche thing that only works well for a very specific workflow. So, I'm glad you built it and it's working out for you (that's the main reason I built this extension: to use it :P), and I'll point folks to your fork if they're looking for this feature :)

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.

3 participants