Skip to content

Conversation

AndrewChubatiuk
Copy link
Contributor

What type of PR is this?

  • support more RQ statuses (STOPPED, CANCELED, DEFERRED, etc)

  • consistent status naming

  • replaced deprecated job result handling methods

  • Refactor

  • Feature

  • Bug Fix

  • New Query Runner (Data Source)

  • New Alert Destination

  • Other

Description

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

Looks like it needs a bit more attention in order to pass CI. 😉

@AndrewChubatiuk AndrewChubatiuk force-pushed the consistent-rq-statuses branch from bca9706 to d228308 Compare April 21, 2024 13:57
@AndrewChubatiuk
Copy link
Contributor Author

Looks like it needs a bit more attention in order to pass CI. 😉

just had to update a test

@justinclift
Copy link
Member

@gaecoli @wlach Eyeballing this, it seems sensible. You're both better at Python than me though, so would either of you be ok to take a quick look over this to make sure there's nothing I missed? 😄

Copy link
Member

@gaecoli gaecoli left a comment

Choose a reason for hiding this comment

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

LGTM!

@justinclift justinclift merged commit 5ac5d86 into getredash:master Apr 24, 2024
@AndrewChubatiuk AndrewChubatiuk deleted the consistent-rq-statuses branch April 24, 2024 04:24
harveyrendell pushed a commit to pushpay/redash that referenced this pull request Jan 8, 2025
* consistent rq status naming and handling

* test fix

* make scheduled and deferred statuses cancelable
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