Page 2 of 6

Re: New Module SunriseSunset

Posted: 06 Aug 2014 17:51
by pofs
n0ahg wrote:Agreed, something weird going on. I even commented on a change made to my DummyDevice 21 days ago and no response. I was happy with the openness but now they seem to have swung the other way.

Thanks for the contributions.
I think it is because JS people are on vacation :)

Re: New Module SunriseSunset - ***Updated Module***

Posted: 09 Aug 2014 15:45
by PoltoS
@islipfd19 good job! why not to make a fork on github and make a pull request?

We will have a look on your both modules and possibly merge them into main branch.

Re: New Module SunriseSunset - ***Updated Module***

Posted: 09 Aug 2014 18:30
by islipfd19
Ok, I already have an account with Github. I'll try to do it this in the coming days.

Any comments or help with the other modules I've provided would greatly be appreciated.

Re: New Module SunriseSunset - ***Updated Module***

Posted: 11 Aug 2014 13:52
by islipfd19
PoltoS wrote:@islipfd19 good job! why not to make a fork on github and make a pull request?

We will have a look on your both modules and possibly merge them into main branch.
@PoltoS

I've forked the project and added the three modules to my fork.

Re: New Module SunriseSunset - ***Updated Module***

Posted: 11 Aug 2014 14:19
by n0ahg
islipfd19 wrote:
PoltoS wrote:@islipfd19 good job! why not to make a fork on github and make a pull request?

We will have a look on your both modules and possibly merge them into main branch.
@PoltoS

I've forked the project and added the three modules to my fork.
Now you need to create a pull request, see steps 5 and 6 in the last section here https://github.com/Z-Wave-Me/home-autom ... stallation

Re: New Module SunriseSunset - ***Updated Module***

Posted: 11 Aug 2014 14:33
by islipfd19
n0ahg wrote:
islipfd19 wrote:
PoltoS wrote:@islipfd19 good job! why not to make a fork on github and make a pull request?

We will have a look on your both modules and possibly merge them into main branch.
@PoltoS

I've forked the project and added the three modules to my fork.
Now you need to create a pull request, see steps 5 and 6 in the last section here https://github.com/Z-Wave-Me/home-autom ... stallation
Thanks! Just did it. I created the pull request for the bleeding edge branch. It should be in this one, right? Or should I be adding it to the master branch?

Re: New Module SunriseSunset - ***Updated Module***

Posted: 11 Aug 2014 14:39
by n0ahg
I have done it on the develop branch in the past. Not sure if all the changes in bleeding-edge will make it in to master but as you only add files and don't modify other files you should be fine and they should be able to merge your changes. Still getting my head around git and github.

Re: New Module SunriseSunset - ***Updated Module***

Posted: 11 Aug 2014 14:45
by islipfd19
n0ahg wrote:I have done it on the develop branch in the past. Not sure if all the changes in bleeding-edge will make it in to master but as you only add files and don't modify other files you should be fine and they should be able to merge your changes. Still getting my head around git and github.
Gotcha, thanks!

Re: New Module SunriseSunset - ***Updated Module***

Posted: 06 Sep 2014 10:41
by ozgreg
Loving this module..

If I can suggest an Option to disable the turning off the switch on sunrise (maybe have a Disable in the drop down list might be a good way to support this..

Thanks for your hard work.. :)

Re: New Module SunriseSunset - ***Updated Module***

Posted: 06 Sep 2014 20:20
by islipfd19
It's most probably possible to do this. Before I put some effort into it, I'd like to understand why you would like this option and the scenario or scenarios you use it in? Knowing this will help me improve it.