Benjamin Mitzkus comments

Results 8 comments of


                                            Benjamin Mitzkus

[bug] incorrect payload for tooltip

struggled with the same problem. I assume you are using multiple datasets/series that you are plotting in the same graph? Try this: ```jsx ... ``` For reference: [codesandbox](https://codesandbox.io/s/async-frog-3lz3om?file=/src/App.js)

fixed wildcard parse bug

I just noticed that there is a PR open for this, however it is from 2019...so I will leave this open as a reminder

Contributions for robustness tests

Hey, thanks for your contribution. Sadly, in the current state this is impossible to review. Please remove all the .jpgs from the merge request and I'll take a look =)

fixed wildcard parse bug

I am sorry @maxshine, my PR was not meant to take your contribution away. I only realised after I'd written the PR that you wrote one with the exact same...

[BUG] - Scheduler.Update() is not working with Cron() & CronWithSeconds()

Hey @JohnRoesler , thanks for taking the time to explain this in detail. I tried to simplify my code to a MWE, but overlooked the pointer issue. However, if I...

[BUG] - Scheduler.Update() is not working with Cron() & CronWithSeconds()

@JohnRoesler `github.com/go-co-op/gocron v1.32.1` on `go 1.18`.

[BUG] - Scheduler.Update() is not working with Cron() & CronWithSeconds()

Open for contributions? I can try to take a look

[BUG] - Scheduler.Update() is not working with Cron() & CronWithSeconds()

The bug seems to be related to this part of the code: https://github.com/go-co-op/gocron/blob/e3b4f5f2e688c5ebb757c2d47ebb743f16d0a6c4/scheduler.go#L205-L225 L224 seems to be the issue here. If I change it to ```golang lastRun = job.LastRun() ```...