MMM-doomsDay - The countdown module,

broberg

@Mykle1 no worries, first versions always have bugs :D

Mykle1

@broberg said in MMM-doomsDay - The countdown module,:

no worries, first versions always have bugs

I see you have a new designation.
Congratulations, Mr. Module Developer!

Mykle1

@broberg said in MMM-doomsDay - The countdown module,:

first versions always have bugs

Just a follow up. I did the git pull to go back to the original version as you advised. I didn’t disable the module but I did set a new doomsDay for Tuesday. (This was yesterday) The module correctly reported “4 Days Left”. but as of this morning, it still reads "4 Days Left.

broberg

@Mykle1 Finally home, and I believe I have fixed this issue now!
do a new git pull and it should work in the mornings as well :D

@illskool You should also do an update on this module! :D

Mykle1

@broberg said in MMM-doomsDay - The countdown module,:

Finally home, and I believe I have fixed this issue now!
do a new git pull and it should work in the mornings as well

Thanks bro. Gonna do that right now.
Ok, I have my test running for tomorrow. The module is correctly displaying “1 Day Left”

{
    			module: 'MMM-doomsDay',
    			position: 'top_right', // This can be any of the regions, best results in center regions
    			config: {
        			doomsDay: "2017-02-27 24:00:00", // the end day of the countdown
        			toWhat: 'Build Bed!', // the title of your countdown event
				present: 'Get to it!',
				singular: 'Day Left', // what it should say when it's only one day left, default is `Day Left`
				plural: 'Days Left', // what it should say when it's more days left, default is `Days Left`
				}
		},

Mykle1

@broberg

It seems I’m still having some problems with this. It worked on the last test, and displayed the present: string correctly, but now it’s not working again. Below is my config with the doomsDay set for tomorrow and a screenshot of the display still reporting 2 Days Left

    			module: 'MMM-doomsDay',
    			position: 'top_right', // This can be any of the regions, best results in center regions
    			config: {
        			doomsDay: "2017-03-01 24:00:00", // the end day of the countdown
        			toWhat: 'Doctor Dave Wed', // the title of your countdown event
				present: 'Doctor Dave 11:30',
				singular: 'Day Left', // what it should say when it's only one day left, default is `Day Left`
				plural: 'Days Left', // what it should say when it's more days left, default is `Days Left`
				}
		},

Mykle1

@Mykle1

Somewhere between now and when I posted my previous comment, the module updated to 1 Day left.

broberg

@Mykle1 I think a higher updateInterval may be in order. the 5 Hours set now maybe is a bit to much,

in the config you can add

updateInterval: 1 * 3600000,

To set it to update every hour instead!

Mykle1

@broberg

Done. I’ll keep you posted. Thanks for your time and effort. :thumbsup_tone1:

broberg

@Mykle1 And thank you for all the beta testing! :D

Mykle1

@broberg said in MMM-doomsDay - The countdown module,:

And thank you for all the beta testing!

My pleasure. Woke up to the module displaying the correct present: entry. Nice!

Mykle1

@broberg

I hate to be the bearer of bad news, but as of this morning, the module is still not performing correctly. Yesterday, I set a simple test for tomorrow. It correctly reported 2 Days Left. This morning (7:09am) it still reports 2 Days Left. I’ve made only the changes that you have advised.

sorry. no screenshot. I’m not at my normal computer

Mykle1

@Mykle1

{
    			module: 'MMM-doomsDay',
    			position: 'top_right', // This can be any of the regions, best results in center regions
    			config: {
        			doomsDay: "2017-03-03 24:00:00", // the end day of the countdown, 
        			toWhat: 'Pay Day is in', // the title of your countdown event
				present: 'Benjamin Pile!',
				updateInterval: 1 * 3600000,
				singular: 'more day!', // what it should say when it's only one day left, default is `Day Left`
				plural: 'more days', // what it should say when it's more days left, default is `Days Left`
				}
		},

Mykle1

@broberg

Just glanced at the mirror and doomsDay is now reporting 1 more day. I don’t know at what time that occurred.

Mykle1

@Mykle1

Reporting the correct present: message this morning. I’ll now set a week long countdown and see how that goes. I’ll try not to bother you anymore

Mykle1

@broberg said in MMM-doomsDay - The countdown module,:

And thank you for all the beta testing!

I know I said I would try not to bother you anymore with this but . . .

I set doomsDay: "2017-03-15 24:00:00", which is the Ides of March, just as a test. I keep testing because I really want this module to work for me. Well, as of 8:50am today, 2017-03-04. the module is reporting 12 More Days instead of 11. The last test didn’t update the Days Left until late morning, early afternoon. I wouldn’t mind if it was slow in updating by an hour or two but it’s no good to me if it’s 12 hours behind

Sorry

broberg

@Mykle1 Okay, do a new git pull now. I have altered the method of rounding the numbers, which I should have done from the beginning if I knew better :P

Anyhow. Try the new version!

Mykle1

@broberg said in MMM-doomsDay - The countdown module,:

Anyhow. Try the new version!

I’ll get it right now. Thanks for dealing with this for me.

Did the git pull

I changed my test to the 7th as a 3 day test. I didn’t want to wait till the 15th to see if it was working correctly. It’s reporting 3 Days Left so that’s an improvement right away. I’ll check it in the morning. Thanks again

strawberry 3.141

@broberg date and time stuff can get really painful, that’s why I’m using moment for those tasks. there are some clever brains that thought about all kind of edge cases etc. you might want to check it out http://momentjs.com/docs/#/displaying/to/

broberg

@strawberry-3.141 Yeah I’ve noticed. If this module keeps misbehaving I will look in to the moment.js instead. looka a lot simpler, thanks for the link!