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 create new task button to top of task history page #995

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

wintonzheng
Copy link
Contributor

@wintonzheng wintonzheng commented Oct 17, 2024

Important

Adds a "Create New Task" button to TaskList.tsx, linking to the /create route.

  • UI Enhancement:
    • Adds "Create New Task" button to TaskList.tsx.
    • Uses Button, Link, and PlusIcon components.
    • Button links to /create route for task creation.

This description was created by Ellipsis for a9c79d9. It will automatically update as commits are pushed.

…src/'

<!-- ELLIPSIS_HIDDEN -->

> [!IMPORTANT]
> Add "Create New Task" button to `TaskList.tsx` for task creation.
>
>   - **UI Enhancement**:
>     - Adds a "Create New Task" button to the top of the task history page in `TaskList.tsx`.
>     - Utilizes `Button`, `Link`, and `PlusIcon` components for the button design and functionality.
>     - Button links to the `/create` route for task creation.
>
> <sup>This description was created by </sup>[<img alt="Ellipsis" src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=Skyvern-AI%2Fskyvern-cloud&utm_source=github&utm_medium=referral)<sup> for c477f0b251f0db5901029a6435a9fdfbb321bf34. It will automatically update as commits are pushed.</sup>

<!-- ELLIPSIS_HIDDEN -->
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to a9c79d9 in 8 seconds

More details
  • Looked at 27 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. skyvern-frontend/src/routes/tasks/list/TaskList.tsx:19
  • Draft comment:
    Ensure the 'asChild' prop in the Button component is necessary. If not, consider removing it for simplicity.
  • Reason this comment was not posted:
    Confidence changes required: 33%
    The header containing the 'Create New Task' button is correctly implemented, but the use of 'asChild' prop in Button component might not be necessary.

Workflow ID: wflow_Z0BBLx35e0A7nVL1


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Incremental review on a9c79d9 in 9 seconds

More details
  • Looked at 27 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. skyvern-frontend/src/routes/tasks/list/TaskList.tsx:19
  • Draft comment:
    Ensure the Button component supports the asChild prop. If not, consider using a different approach to nest the Link component inside the Button.
  • Reason this comment was not posted:
    Comment did not seem useful.

Workflow ID: wflow_CoQ1xgqoUeEADqZn


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@msalihaltun msalihaltun merged commit 7802f8e into main Oct 17, 2024
2 checks passed
@msalihaltun msalihaltun deleted the salih/add-create-task-button-to-task-history-page branch October 17, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants