Conversation
Implement the ability to return index and error HTML pages. Implement the ability to 'sign up' by receiving a unique user ID, provide and revoke cookie consents. Create index.html and error.html (both contain placeholders / 'lorem ipsum' at the moment. Create a random string generator.
Supply static pages, assign / delete users.
Fix Github runner for debugging Java code.
Implement backend static functionality and cookies.
Add new team member's details.
- Added the Chatbot Interaction Flow Diagram - A bit of formatting fixes
Uploaded 3 versions of the Chatbot Interaction Flow diagram to docs.
Changed the link of watson_flow.png from https://github.com... to /docs/watson_flow.png
Remove "frontend" from the branch list.
Add the client-chatbot interaction flowchart.
Revert "Supply static pages, assign / delete users." Revert the previous incorrect merge.
Add information about Siyuan - the new team member.
GitHub maven runner fix
Add IBMAuthenticator.java and IBMAuthenticatorTest.java . IBMAuthenticator uses IBM IAM API key to receive temporary bearer tokens. Bearer tokens are used to make requests to Watsonx API.
Add WatsonxAPIController.java, WatsonxResponse.java. Rename RandomString.java to SpringTools.java, add method for preparing input strings. Refactor IBMAuthenticator.java . The code can now make requests to Watsonx API.
User.java now stores the value of consent to optional cookies. Add UserTest.java - unit tests for 'User' class.
Chat.java stores information about the chat - its owner and message history. It allows to add the user's and the AI's messages to its history. ChatTest.java contains unit tests for the Chat class.
DatabaseController now has getUser method, improved addUser and removeUser methods. DatabaseController now has new createChat, deleteChat and getChat methods. Add DatabaseController unit tests. SpringTools.messageHistoryPrepare now appends 'assistant' keyword before replacements.
Hardcode the correct system prompt to Chat.java . Update ChatTest and WatsonxAPIControllerTest .
Improve signup and revokeConsent methods. Add createChat, sendMessage, sendIncognitoMessage, getChatHistory methods.
Due to issues with IBM APIs, temporary disable WatsonxAPIControllerTest tests.
Add backendHTTPMethods flowchart; Add userRegistrationFlowchart flowchart; Both flowcharts are saved to 'docs' directory.
feature/issue-46/watsonx-integration
Move backend files out of 'backend' directory into /; Remove 'backend' directory; Edit the Maven Github workflow runner (runs only on 'dev' branch now); Adjust .gitignore.
Remove 'Dependency graph' from maven.yml runner; Dependency graphs are not supported in our repository.
feat: Add unit tests; integrate Watsonx API.
chore: Add PR template.
Separate building and testing
RainBOY-ZZX
previously approved these changes
Jan 15, 2025
vladislav-k1
requested changes
Jan 15, 2025
Contributor
vladislav-k1
left a comment
There was a problem hiding this comment.
Change the comments to English, everything else looks well.
RainBOY-ZZX
previously approved these changes
Jan 15, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue(s):
#110
Type of change: (choose required ones)
Description:
fixing the optional cookie part
Additional context:
fix the interface
Testing instructions:
connect to the backend and got cookie date