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

Jump to bottom

[UI v2] feat: Adds deployment details RQ hook #16994

Merged
merged 1 commit into from
Feb 6, 2025
Merged

[UI v2] feat: Adds deployment details RQ hook #16994

merged 1 commit into from
Feb 6, 2025

Conversation

devinvillarosa
Copy link
Contributor

@devinvillarosa devinvillarosa commented Feb 5, 2025 •
edited
Loading

  1. Adds RQ hook for getting a deployment's details based on its id
  2. Ensures loader based on QueryOptions from above for the deployment details page

Checklist

  • This pull request references any related issue by including "closes <link to issue>"
    • If no issue exists and your change is not a small fix, please create an issue first.
  • If this pull request adds new functionality, it includes unit tests that cover the changes
  • If this pull request removes docs files, it includes redirect settings in mint.json.
  • If this pull request adds functions or classes, it includes helpful docstrings.

Relates to #15512

@github-actions github-actions bot added the ui-replatform Related to the React UI rewrite label Feb 5, 2025
@devinvillarosa devinvillarosa force-pushed the add-deployment-details-rq branch from a07f794 to f122bb6 Compare February 5, 2025 21:00
@devinvillarosa devinvillarosa force-pushed the add-deployment-details-rq branch from f122bb6 to a5bfe3d Compare February 6, 2025 18:09
@devinvillarosa devinvillarosa merged commit cf1e18d into main Feb 6, 2025
11 checks passed
@devinvillarosa devinvillarosa deleted the add-deployment-details-rq branch February 6, 2025 18:32
kevingrismore pushed a commit that referenced this pull request Feb 7, 2025
@devinvillarosa @kevingrismore
[UI v2] feat: Adds deployment details RQ hook (#16994)
23ffbcc
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@pleek91 pleek91 pleek91 approved these changes

@aaazzam aaazzam Awaiting requested review from aaazzam aaazzam is a code owner

@desertaxle desertaxle Awaiting requested review from desertaxle desertaxle is a code owner

@devangrose devangrose Awaiting requested review from devangrose devangrose is a code owner

Assignees
No one assigned
Labels
ui-replatform Related to the React UI rewrite
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@devinvillarosa @pleek91