Skip to content

Question validation disable option#767

Merged
yangcao77 merged 12 commits into
redhat-developer:mainfrom
yangcao77:question-validation-disable-option
May 13, 2025
Merged

Question validation disable option#767
yangcao77 merged 12 commits into
redhat-developer:mainfrom
yangcao77:question-validation-disable-option

Conversation

@yangcao77
Copy link
Copy Markdown
Contributor

Hey, I just made a Pull Request!

for issue https://issues.redhat.com/browse/RHDHPAI-759

add optional boolean plugin configuration disableQuestionValidation

also due to the change in road-core, road-core/service#567
we are now providing all RHDH customized prompt in road-core
thus, remove the rhdh prompt from lightspeed plugin.

to test locally, the simplest way is change the value from ols to rhdh before start rcs locally: https://github.com/road-core/service/blob/b06b739bf6eefd42fe6838ef9c5e758e34323b62/ols/customize/__init__.py#L6
or if use rhdh image, provide PROJECT=rhdh ENV when start the container
e.g.

podman run -it --rm -v ./test-config:/app-root/config:Z  -e RCS_CONFIG_FILE=/app-root/config/rcsconfig.yaml -e PROJECT=rhdh -p 8080:8080 <image>

✔️ Checklist

  • A changeset describing the change and affected packages. (more info)
  • Added or Updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

@rhdh-gh-app
Copy link
Copy Markdown

rhdh-gh-app Bot commented May 12, 2025

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-lightspeed-backend

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-lightspeed-backend workspaces/lightspeed/plugins/lightspeed-backend none v0.5.1

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@karthikjeeyar karthikjeeyar left a comment

Choose a reason for hiding this comment

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

Verified locally, works fine

image

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label May 13, 2025
@yangcao77 yangcao77 merged commit 650ad75 into redhat-developer:main May 13, 2025
9 checks passed
elai-shalev pushed a commit to elai-shalev/rhdh-plugins that referenced this pull request Jan 21, 2026
* add RHDH system prompt

Signed-off-by: Stephanie <yangcao@redhat.com>

* add optional boolean value to config

Signed-off-by: Stephanie <yangcao@redhat.com>

* remove system prompt from lightspeed plugin

Signed-off-by: Stephanie <yangcao@redhat.com>

---------

Signed-off-by: Stephanie <yangcao@redhat.com>
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