Vba code to cancel a timer

Hi,

I have a worksheet with a pop up macro, now i need the code to stop the macro.

my sheet has 10 rows (each row for a different contractor), so when a contractor calls me i enter his details and start the macro by clicking a button, when i run the macro my Msgbox appears every hour, i would now like to know how to create a macro to stop the Msgbox appearing. i am using the code below but it doesnt work.

Please Login or Register  to view this content.

thanks in advance

You have code to stop the timer in two different places. Let's just use the one in Module1 and ignore the one in Module2.

The call to stop the timer must have the time that the timer is set for. You are using interval for this, but the variable interval is never declared and never set, so it's going to be 0. I strongly recommend to everyone that they use Option Explicit and declare variables. Doing so prevents a lot of bugs, like this one.

You are starting the timer to expire in 10 seconds. This does not match your description of 1 hour, so I'm not clear on what you are trying to do.

There is the potential to start multiple timers. So to cancel any of them, you have to store the time someplace for each one. I am going to suggest you do this in column AK. However, that also means you have to determine which row is affected when the timer button is clicked.

I get an error on the call to LockCells. That is because you are using merged cells. Merged cells are almost always a bad idea that will cause problems, especially with VBA. I fixed the code but I strongly suggest you redesign so you don't need merged cells.

I only updated the buttons in the first row. You will have to set all Start buttons to call My_Macro and all Complete buttons to call stop_macro.

I suspect that after you get the timer issue settled you will see other problems. Post back to this same thread when that happens.

@GReg M this was my initial thought as well but this method will not work because there could be multiple timers active at one time, so the solution must keep track of multiple EarliestTime values.

Attached a different approach to timing - each channel has its own 'ticker' so they do not interact/interfere with each other
Use for split work timing (solicitor/accountant) - some of the coding methodology I have used maybe useful.