Skip to content
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

Open
cforce opened this issue Oct 2, 2013 · 5 comments
Open

same as due_date_reminder, differences , merge? #12

cforce opened this issue Oct 2, 2013 · 5 comments

Comments

@cforce
Copy link

cforce commented Oct 2, 2013

I am sking where are the differences and how to merge both in one solution ..

f0y/due_date_reminder#40

@Enkil
Copy link
Owner

Enkil commented Oct 2, 2013

differences are in readme and in last merge from JostBaron

@f0y
Copy link

f0y commented Jul 2, 2014

Hi! I am author of due date reminder. What was a point in making the fork?

@Silex
Copy link

Silex commented Sep 23, 2014

The fork looks rather ridiculous: https://github.com/Enkil/redmine_reminder/blob/master/init.rb#L38

Didn't even rename correctly...

@DarkSideGeek
Copy link

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...

@cforce
Copy link
Author

cforce commented Jan 31, 2016

Best of both - work together-that's github ;)

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

No branches or pull requests

5 participants