I think it is because JS people are on vacationn0ahg 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.
[userModule] SunriseSunset - No Longer Maintained
Re: New Module SunriseSunset
Re: New Module SunriseSunset - ***Updated Module***
@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.
We will have a look on your both modules and possibly merge them into main branch.
Re: New Module SunriseSunset - ***Updated Module***
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.
Any comments or help with the other modules I've provided would greatly be appreciated.
Re: New Module SunriseSunset - ***Updated Module***
@PoltoSPoltoS 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.
I've forked the project and added the three modules to my fork.
Re: New Module SunriseSunset - ***Updated Module***
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 ... stallationislipfd19 wrote:@PoltoSPoltoS 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.
I've forked the project and added the three modules to my fork.
Re: New Module SunriseSunset - ***Updated Module***
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?n0ahg wrote: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 ... stallationislipfd19 wrote:@PoltoSPoltoS 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.
I've forked the project and added the three modules to my fork.
Re: New Module SunriseSunset - ***Updated Module***
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***
Gotcha, thanks!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.
Re: New Module SunriseSunset - ***Updated Module***
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..
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***
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.