Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[FIX] -- job status display in scheduler listing page template. #694

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DhavalGojiya
Copy link

Fix job status display in scheduler template

Description

This PR updates scheduler.html to correctly display the job status by using job.get_status.value instead of job.get_status.

Changes

  • Updated job status display in scheduler.html.
  • Affects the scheduler view at /django-rq/schedulers/<scheduler_id>/.

Before Screenshot:

Before PR

After Screenshot:

After PR

Testing

  • Verified that the job status is displayed correctly in the scheduler view.
  • Ensured no other UI elements were affected.

- Updated 'job.get_status' to 'job.get_status.value' in 'scheduler.html' to correctly display the job status.
- This affects the scheduler view at '/django-rq/schedulers/<scheduler_id>/'.
@@ -93,7 +93,7 @@
{{ job.ended_at|to_localtime|date:"Y-m-d, H:i:s" }}
{% endif %}
</td>
<td>{{ job.get_status }}</td>
<td>{{ job.get_status.value }}</td>

Choose a reason for hiding this comment

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

please fix test cases as well

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

Successfully merging this pull request may close these issues.

2 participants