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

scheduler: improvments #2937

Merged
merged 1 commit into from
Jun 21, 2022
Merged

scheduler: improvments #2937

merged 1 commit into from
Jun 21, 2022

Conversation

rerowep
Copy link
Contributor

@rerowep rerowep commented May 31, 2022

  • Uses f strings for printing.
  • Adds replace-idby-subjects-imported scheduled task.

Co-Authored-by: Peter Weber peter.weber@rero.ch

Why are you opening this PR?

  • Which task/US does it implement?
  • Which issue does it fix?

Dependencies

My PR depends on the following rero-ils-ui's PR(s):

  • rero/rero-ils-ui#

How to test?

  • What command should I have to run to test your PR?
  • What should I test through the UI?

@rerowep rerowep self-assigned this May 31, 2022
@rerowep rerowep force-pushed the wep-better-scheduler branch 6 times, most recently from 3fbcc85 to ee522b9 Compare June 8, 2022 05:16
@rerowep rerowep requested a review from BadrAly June 8, 2022 05:23
@rerowep rerowep force-pushed the wep-better-scheduler branch 2 times, most recently from 63281f2 to cf42ba6 Compare June 8, 2022 13:09
@rerowep rerowep force-pushed the wep-better-scheduler branch from cf42ba6 to aec7fc4 Compare June 10, 2022 12:39
* Uses f strings for logging.
* Adds `replace-idby-subjects-imported` scheduled task.

Co-Authored-by: Peter Weber <peter.weber@rero.ch>
@rerowep rerowep force-pushed the wep-better-scheduler branch from aec7fc4 to b54696c Compare June 13, 2022 13:50
@rerowep rerowep merged commit 9b3eee5 into rero:staging Jun 21, 2022
@rerowep rerowep deleted the wep-better-scheduler branch July 25, 2022 05:59
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.

3 participants