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

Add rq dashboard for local development #7108

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

zachliu
Copy link
Contributor

@zachliu zachliu commented Aug 5, 2024

What type of PR is this?

  • Refactor
  • Feature
  • Bug Fix
  • New Query Runner (Data Source)
  • New Alert Destination
  • Other

Description

Add the RQ dashboard in docker compose for better local development.

rq-dashboard-UI

rq-dashboard-workers

rq-dashboard-finished

How is this tested?

  • Unit tests (pytest, jest)
  • E2E Tests (Cypress)
  • Manually
  • N/A

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

@justinclift
Copy link
Member

This seems like a really good idea.

We should add some useful mention/docs about it in appropriate places too. eg maybe a new wiki page to start with, and then add something to the website


As this is a significant item / change instead of just approving and merging it I'd like to get the opinion of others as well. @eradman, what do you reckon? 😄

@eradman
Copy link
Collaborator

eradman commented Oct 22, 2024

Doesn't this provide the same functionality as the "RQ Status" Admin page? If there is missing functionality, I think I would rather see the built-in Status page improved.

admin-rq

@zachliu
Copy link
Contributor Author

zachliu commented Oct 22, 2024

I consider it as a "stand-in replacement" while the "RQ Status" page is being improved
It's just that i don't see anyone is actively working on improving that page 😛

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