Skip to content

Add TickerQ Background Worker Integration #23802

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.

Sign up for GitHub

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

Jump to bottom
Merged
maliming merged 26 commits into from
Oct 7, 2025
Merged

Add TickerQ Background Worker Integration #23802

maliming merged 26 commits into from
Oct 7, 2025
+1,210 −8

Conversation

Copy link
Contributor

Copilot AI commented Sep 28, 2025 •
edited by maliming
Loading

Fixes #23426

image image image

Copilot AI changed the title [WIP] Is there any plan to integrate TickerQ? Add TickerQ Background Worker Integration Sep 28, 2025
Copilot AI requested a review from hikalkan September 28, 2025 19:02
@maliming maliming requested review from maliming and removed request for hikalkan September 29, 2025 01:01
@maliming maliming added this to the 10.1-preview milestone Sep 29, 2025
@maliming maliming marked this pull request as ready for review September 29, 2025 05:19
@maliming
Copy link
Member

maliming commented Sep 29, 2025

@copilot Can you add net10.0 after net9.0 in your csproj files?

Copy link
Contributor Author

Copilot AI commented Sep 29, 2025

@copilot Can you add net10.0 after net9.0 in your csproj files?

Added net10.0 after net9.0 in both csproj files to align with other ABP projects. Changes in commit 4d9b25e.

@codecov
Copy link

codecov bot commented Oct 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.09%. Comparing base (019ac06) to head (2750b3a).
⚠️ Report is 9 commits behind head on dev.

Additional details and impacted files 
@@           Coverage Diff           @@
##              dev   #23802   +/-   ##
=======================================
  Coverage   51.09%   51.09%           
=======================================
  Files        3269     3269           
  Lines      106431   106432    +1     
  Branches     8173     8173           
=======================================
+ Hits        54378    54379    +1     
  Misses      50379    50379           
  Partials     1674     1674

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@maliming maliming mentioned this pull request Oct 5, 2025
Closed
1 task
@abpframework abpframework deleted a comment from CLAassistant Oct 6, 2025
@CLAassistant
Copy link

CLAassistant commented Oct 6, 2025 •
edited
Loading

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ maliming
✅ EngincanV
❌ Copilot
You have signed the CLA already but the status is still pending? Let us recheck it.

maliming and others added 3 commits October 6, 2025 10:01
@maliming
Handling conflicts.
c1b6557
@EngincanV
Update docs-nav.json
6aa0528
@EngincanV
Fix path separator in solution file and improve docs
76300f4 
Replaced backslash with forward slash in the project path for TickerQ demo app in the solution file to ensure consistency across platforms. Also clarified documentation on configuring TimeTicker properties for specific jobs.
hikalkan
hikalkan requested changes Oct 7, 2025
View reviewed changes
@maliming
Copy link
Member

maliming commented Oct 7, 2025

@copilot, can you sign the CLA? https://cla-assistant.io/abpframework/abp?pullRequest=23802

@maliming maliming merged commit 98ff26d into dev Oct 7, 2025
2 of 4 checks passed
@maliming maliming deleted the copilot/fix-d5496b38-0efc-4016-8ab2-970b67675314 branch October 7, 2025 08:36
Copilot AI requested a review from maliming October 7, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EngincanV EngincanV EngincanV approved these changes

@maliming maliming Awaiting requested review from maliming

+2 more reviewers

@rqx110 rqx110 rqx110 left review comments

@hikalkan hikalkan hikalkan approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

Copilot code review Copilot

Labels

abp-framework feature

Projects

None yet

Milestone

10.1-preview

Development

Successfully merging this pull request may close these issues.

Is there any plan to integrate TickerQ?

6 participants

@maliming @CLAassistant @hikalkan @rqx110 @EngincanV