Quartz XML docs still describe only jobs with SentryMonitorSlug
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 89/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- csharp
- Domain
- documentation
Research direction
Read the XML comments in src/Sentry.Quartz3/SentryJobListener.cs and src/Sentry.Quartz/Core/SentryQuartzOptions.cs, then compare them with the AddSentry documentation mentioned in the issue. Update both comments to account for jobs monitored through MonitorAllJobs; done when neither comment implies monitoring is limited to jobs with SentryMonitorSlugAttribute.
Written by the indexing model from the issue text.
Description
#5741 added SentryQuartzOptions.MonitorAllJobs, but two XML doc comments still say only jobs with [SentryMonitorSlug] are monitored:
SentryJobListener(Sentry.Quartz3): "sends a Sentry check-in each time a job withSentryMonitorSlugAttributeruns". WithMonitorAllJobsit also sends them for other scheduled jobs.SentryQuartzOptions.SendMonitorConfig: "the in-progress check-in of each job withSentryMonitorSlugAttributeincludes the schedule…". It also applies to jobs monitored throughMonitorAllJobs.
The docs on SentryQuartzBuilderExtensions.AddSentry already mention MonitorAllJobs and can be used as the model.
Found while checking the docs PR getsentry/sentry-docs#20051 against main.
- Dominant language
- C#
- Stars
- 772
- Forks
- 256
- Avg merge
- 1d 22h
- Merged PRs (30d)
- 102
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from getsentry/sentry-dotnet
-
Bug Platform: Android State: Needs Discussion
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
getsentry/sentry-dotnet#5764 · 1 comment ·
Maintainers usually reply within 1 day
-
docs: list supported versions in the MAUI guide and remove TizenPossibly taken @ric-oliv claimed this 15 days ago. Open.NET Docs Framework: MAUI
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
getsentry/sentry-dotnet#5714 · 2 comments · 1 assignee ·
Maintainers usually reply within 1 day
-
Document the release and environment changes for v7.0.0Possibly taken @jamescrosswell claimed this 16 days ago. Open.NET Docs Next Major
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
getsentry/sentry-dotnet#5709 · 1 comment · 1 assignee ·
Maintainers usually reply within 1 day
-
Docs MSBuild Needs Docs Update
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
getsentry/sentry-dotnet#5691 · 1 comment ·
Maintainers usually reply within 1 day
-
.NET Docs State: Needs Discussion
Difficulty 1/5 Under an hour Newbie friendliness 82/100
getsentry/sentry-dotnet#5637 · 1 comment ·
Maintainers usually reply within 1 day
All issues in getsentry/sentry-dotnet
Similar issues
-
bug effort:S P3
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
nightscout/nocturne#3093 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
Maintainers usually reply within 3 days
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100