-
-
Notifications
You must be signed in to change notification settings - Fork 269
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FEATURE: [xnav] add cron schedule #1470
Conversation
Welcome back! @narumiruna, This pull request may get 231 BBG. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1470 +/- ##
=======================================
Coverage 21.32% 21.32%
=======================================
Files 591 591
Lines 43482 43482
=======================================
Hits 9274 9274
Misses 33533 33533
Partials 675 675 Continue to review full report in Codecov by Sentry.
|
pkg/strategy/xnav/strategy.go
Outdated
go func() { | ||
ticker := time.NewTicker(util.MillisecondsJitter(s.Interval.Duration(), 1000)) | ||
defer ticker.Stop() | ||
if s.Schedule != "" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be one of "Schedule" or "Interval" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
ad90f37
to
7e07f58
Compare
Re-estimated karma: this pull request may get 259 BBG |
7e07f58
to
3d2b888
Compare
Re-estimated karma: this pull request may get 263 BBG |
3d2b888
to
5592d93
Compare
Re-estimated karma: this pull request may get 268 BBG |
Hi @narumiruna, Well done! 293 BBG has been sent to your polygon wallet. Please check the following tx: https://polygonscan.com/tx/0x98545d54d5bec0b71b64ba31f56861013d30eac283b415cf1d26c925edddc620 Thank you for your contribution! |
No description provided.