Skip to content

Conversation

misspia-cohere
Copy link
Contributor

@misspia-cohere misspia-cohere commented Jun 24, 2024

Thank you for contributing to the Cohere Toolkit!

Screenshot 2024-06-24 at 18 09 58

  • PR title: "area: description"

    • Where "area" is whichever of interface, frontend, model, tools, backend, etc. is being modified. Use "docs: ..." for purely docs changes, "infra: ..." for CI changes.
    • Example: "deployment: add Azure model option"
  • PR message: Delete this entire checklist and replace with

    • Description: a description of the change
    • Issue: the issue # it fixes, if applicable
    • Dependencies: any dependencies required for this change
  • Add tests and docs: Please include testing and documentation for your changes

  • Lint and test: Run make lint and make run-tests

AI Description

This pull request makes changes to the AgentForm component in the src/interfaces/coral_web/src/components/Agents/AgentForm.tsx file.

Summary

The PR introduces a new RequiredInputLabel component that extends the existing InputLabel component. This new component enhances the user experience by clearly indicating which input fields are mandatory.

Changes

  • Renames the InputLabel component to RequiredInputLabel when displaying the "name" field.
  • Adds a new RequiredInputLabel component that:
  • Accepts label, children, and className props.
  • Renders the label with a flex layout, including the label text and a "required" indicator.

@misspia-cohere misspia-cohere changed the title coral-web: Add required label to agent form coral-web: Add required field label to agent form Jun 24, 2024
@misspia-cohere misspia-cohere merged commit bf09008 into main Jun 25, 2024
1 check failed
@misspia-cohere misspia-cohere deleted the pia/create-agent-required-fields branch June 25, 2024 17:36
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.

2 participants