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

132 adding fallling back to human librarians in case chatbot throws errors #146

Conversation

dominhnhut01
Copy link
Collaborator

No description provided.

@dominhnhut01 dominhnhut01 requested a review from Lamp-Nguyen May 3, 2024 18:14
@dominhnhut01 dominhnhut01 self-assigned this May 3, 2024
@dominhnhut01 dominhnhut01 requested a review from TranHVLoc May 3, 2024 18:15
@dominhnhut01 dominhnhut01 changed the title 132 adding fallling back to human librarians in case chatbot throws errors2 132 adding fallling back to human librarians in case chatbot throws errors May 3, 2024
dominhnhut01 and others added 7 commits May 10, 2024 14:19
* fix cache issue on deployment
* Change dockerfiles and other pipeline files to make images lighter and more efficient

---------

Co-authored-by: Loc Tran <[email protected]>
* - Update CMD in Dockerfile to start the app using serve
- Remove redundant code and files

* Change dockerfiles and other pipeline files to make images lighter and more efficient

* Add entrypoint shell script so containers can read secret at runtime

---------

Co-authored-by: Loc Tran <[email protected]>
* - Update CMD in Dockerfile to start the app using serve
- Remove redundant code and files

* Change dockerfiles and other pipeline files to make images lighter and more efficient

* Add entrypoint shell script so containers can read secret at runtime

* Add entrypoint to frontend to build frontend image successfully

---------

Co-authored-by: Loc Tran <[email protected]>
* - Update CMD in Dockerfile to start the app using serve
- Remove redundant code and files

* Change dockerfiles and other pipeline files to make images lighter and more efficient

* Add entrypoint shell script so containers can read secret at runtime

* Add entrypoint to frontend to build frontend image successfully

* Add entrypoint to frontend for build stage

* Add build arguments for frontend Dockerfile

---------

Co-authored-by: Loc Tran <[email protected]>
* - Update CMD in Dockerfile to start the app using serve
- Remove redundant code and files

* Change dockerfiles and other pipeline files to make images lighter and more efficient

* Add entrypoint shell script so containers can read secret at runtime

* Add entrypoint to frontend to build frontend image successfully

* Add entrypoint to frontend for build stage

* Run the entrypoint before exposing the app port in frontend.dockerfile

* Add build arguments for frontend Dockerfile

* Adding "serve". Modifies the main.ts file to enable CORS with specific origin, methods, and credentials. vite.config.js file is updated to include a proxy configuration for the '/socket.io/' path.

* Update SocketContextProvider to disconnect socket on cleanup. It modifies component to disconnect the socket when the component is unmounted. Also disable CORS in backend since it's still connected to frontend normally after problem is resolved

---------

Co-authored-by: Loc Tran <[email protected]>
Meng-V
Meng-V previously approved these changes Jun 13, 2024
TranHVLoc
TranHVLoc previously approved these changes Jun 27, 2024
@TranHVLoc TranHVLoc dismissed stale reviews from Meng-V and themself via 2c576c2 July 27, 2024 18:44
@TranHVLoc
Copy link
Collaborator

@Meng-V please pull it down and test it with your prompt. Try to crash the chatbot to test if the chat history has been passed successfully to LibrarianWidgetForm. If you can see it, when we can approve the PR

TranHVLoc
TranHVLoc previously approved these changes Nov 6, 2024
Copy link
Collaborator

@TranHVLoc TranHVLoc left a comment

Choose a reason for hiding this comment

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

Lgtm!

@TranHVLoc TranHVLoc self-assigned this Nov 7, 2024
Lamp-Nguyen
Lamp-Nguyen previously approved these changes Nov 7, 2024
@TranHVLoc TranHVLoc dismissed stale reviews from Lamp-Nguyen and themself via fb85796 November 22, 2024 06:39
@TranHVLoc
Copy link
Collaborator

@Lamp-Nguyen can you please review the PR again?

@TranHVLoc TranHVLoc merged commit 36c3b0f into dev Nov 25, 2024
4 checks passed
@TranHVLoc TranHVLoc deleted the 132-adding-fallling-back-to-human-librarians-in-case-chatbot-throws-errors2 branch November 25, 2024 18:24
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.

4 participants