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

feat(chat): allow additional types for custom buttons (Issue #3270) #3293

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

Gimir
Copy link
Contributor

@Gimir Gimir commented Mar 4, 2025

Description:

  • add additional from value types

Issues:

Checklist:

  • the pull request name complies with Conventional Commits
  • the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
  • the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
  • I confirm that do not share any confidential information like API keys or any other secrets and private URLs

@Gimir
Copy link
Contributor Author

Gimir commented Mar 4, 2025

/deploy-review

Environment URL: https://chat-ai-dial-chat-pr-3293.nightly-test.deltixhub.io
E2E tests status: success

@@ -18,13 +18,21 @@ export interface FormSchemaButtonOption {
};
}

export type MessageFormValueType = string[] | number | undefined;
export type MessageFormValueType =
| string[]
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we really need string[]? we was talking only about base types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request _priority_2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants