Skip to content

Fix typos in DBQna README #2047

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

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

Conversation

PeterYang12
Copy link
Contributor

Description

Fix typos in DBQna README

Issues

N/A

Signed-off-by: PeterYang12 <[email protected]>
@Copilot Copilot AI review requested due to automatic review settings June 9, 2025 06:14
Copy link

github-actions bot commented Jun 9, 2025

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

Copy link
Contributor

@Copilot 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 addresses typos in the DBQna README by correcting the Docker image names used in the documentation.

  • Updated the docker build command to tag the text2sql image as "latest" instead of "comps".
  • Corrected the React UI image reference from "dbqna-react-ui:latest" to "text2sql-react-ui:latest" in both image listing and container run commands.
Comments suppressed due to low confidence (3)

DBQnA/docker_compose/intel/cpu/xeon/README.md:14

  • The image tag has been updated to 'latest'; please verify that this change aligns with the established naming conventions for Docker images.
docker build --no-cache -t opea/text2sql:latest -f comps/text2sql/src/Dockerfile .

DBQnA/docker_compose/intel/cpu/xeon/README.md:31

  • The React UI image reference has been corrected from 'dbqna-react-ui' to 'text2sql-react-ui'; please ensure this update is consistent with related configurations.
2. `opea/text2sql-react-ui:latest`

DBQnA/docker_compose/intel/cpu/xeon/README.md:126

  • Since the Docker image name has been updated to 'text2sql-react-ui:latest', consider renaming the container 'test-dbqna-react-ui-server' for consistency if applicable.
docker run -d --name="test-dbqna-react-ui-server" --ipc=host -p 5174:80 -e no_proxy=$no_proxy -e https_proxy=$https_proxy -e http_proxy=$http_proxy opea/text2sql-react-ui:latest

@ZePan110
Copy link
Collaborator

ZePan110 commented Jun 9, 2025

Just remind
I have a PR for the DBQnA documents refine and according to Guidelines and templates from @preethivenkatesh will remove that section. #2034 will remove this part.

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