-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
same as due_date_reminder, differences , merge? #12
Comments
differences are in readme and in last merge from JostBaron |
Hi! I am author of due date reminder. What was a point in making the fork? |
The fork looks rather ridiculous: https://github.com/Enkil/redmine_reminder/blob/master/init.rb#L38 Didn't even rename correctly... |
For those who didn't RTFReadme, the @Enkil version builds upon the @f0y original by adding the ability to send reminders based on a Days-Before schedule defined within individual Issues, per Issue Category defined in a Project, per User preference, and per Global default (in that order). It also looks like the backwards compatibility for 1.x was dropped? If the latter is not a problem, why not do a Pull request and consolidate? IMHO, the fork is not too divergent (yet) and these are worthwhile additions... |
Best of both - work together-that's github ;) |
I am sking where are the differences and how to merge both in one solution ..
f0y/due_date_reminder#40
The text was updated successfully, but these errors were encountered: