Skip to content
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

Move "Periodic Jobs" section out of README into guide #1175

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

whatyouhide
Copy link
Contributor

This is the start of some work on #850. We'll want to split the README out into guides, and the cron plugin seemed like a good candidate to start off—it's isolated and its functionality is self-contained.

Each part may have multiple rules, where rules are separated by a comma. The allowed values for
each field are as follows:

| Field | Allowed Values |
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Turned some stuff into tables here, I think it looks great.

Uploading CleanShot 2024-11-07 at 17.28.34@2x.png…


This plugin registers workers a cron-like schedule and enqueues jobs automatically. Periodic
jobs are declared as a list of `{cron, worker}` or `{cron, worker, options}` tuples.
Periodically enqueue jobs through [cron](https://en.wikipedia.org/wiki/Cron)-based scheduling.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved some stuff around here, I like the flow now, thoughts?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, the new flow is an improvement!

Copy link
Member

@sorentwo sorentwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A fantastic start! Everything you've done here is spot on and the direction we're hoping to go. I've left a few comments about outdated information that was ported from the README. This is ready to go once that's corrected/updated 🎉

guides/periodic_jobs.md Outdated Show resolved Hide resolved
guides/periodic_jobs.md Outdated Show resolved Hide resolved
@whatyouhide
Copy link
Contributor Author

@sorentwo updated ✅

@whatyouhide whatyouhide requested a review from sorentwo November 11, 2024 07:45
guides/periodic_jobs.md Outdated Show resolved Hide resolved
@sorentwo sorentwo merged commit 7c326ce into oban-bg:main Nov 11, 2024
2 checks passed
@sorentwo
Copy link
Member

An excellent start, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants