Skip to content

Conversation

@ri-pandey
Copy link
Contributor

Description

  • Added Workflows-search by Workflow names
  • Changed Workflow-search's Dataset-search field an Autocomplete.

Changes Made

List the main changes made in this PR. Be as specific as possible.

  • Feature added
  • Bug fixed
  • Code refactored
  • Tests changed
  • Documentation updated
  • Other changes: [describe]

Checklist

Before submitting this PR, please make sure that:

  • Your code passes linting and coding style checks.
  • Documentation has been updated to reflect the changes.
  • You have reviewed your own code and resolved any merge conflicts.
  • You have requested a review from at least one team member.
  • Any relevant issue(s) have been linked to this PR.

Additional Information

Corresponding PR for Rhythm API: IUSCA/rhythm_api#6

* implemented search by Workflow name
* made Dataset-select for Workflow search an autocomplete
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements search functionality for workflows by workflow names and enhances the dataset search with an autocomplete component. The changes enable users to search for workflows using workflow names in addition to existing search options.

  • Added a new API endpoint to fetch available workflow names
  • Enhanced the workflow search component to support workflow name selection via dropdown
  • Replaced the dataset name search input with an autocomplete component for better UX

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
ui/src/services/workflow.js Added getWorkflowNames method and workflow_name parameter to getAll function
ui/src/components/runs/WorkflowSearchInputFilter.vue Enhanced search component with workflow name dropdown and dataset autocomplete
api/src/services/workflow.js Added workflow_name parameter to getAll function
api/src/routes/workflows.js Added /names endpoint and workflow_name query parameter validation
ui/src/services/utils.js Fixed trailing comma formatting
ui/src/services/datetime.js Fixed trailing comma formatting
api/src/routes/datasets/uploads.js Fixed missing comma in object property

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@IUSCA IUSCA deleted a comment from Copilot AI Sep 17, 2025
@IUSCA IUSCA deleted a comment from Copilot AI Sep 17, 2025
@IUSCA IUSCA deleted a comment from Copilot AI Sep 17, 2025
@ri-pandey ri-pandey merged commit c458df4 into main Sep 23, 2025
@ri-pandey ri-pandey deleted the feature/workflow-name-search branch September 23, 2025 17:33
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