fix(chat): Resolve knowledge card display and section regeneration bugs
Created by: Edouard-Legoupil
This commit addresses two distinct issues in the Chat.jsx component and the backend API:
-
Display All Outcome Knowledge Cards: The logic for fetching and displaying knowledge cards associated with multiple outcomes has been corrected in
Chat.jsx. Previously,URLSearchParamswas used incorrectly, preventing multipleoutcome_ids from being sent to the backend. The code has been updated to manually construct the query string, ensuring all relevant knowledge cards are fetched and displayed. -
Fix Section Regeneration: The section regeneration feature was failing with a
400 Bad Requesterror because the temporary session created for the regeneration process was missing theproposal_template. The/regenerate_section/{proposal_id}endpoint inbackend/api/proposals.pyhas been modified to load the proposal's template from the database and include it in the temporary Redis session. This resolves the error and restores the functionality.
- Does my code meet the quality standards for releasing packages?
- Does the reviewer have all the information to validate the features/issues without too much research?
- Does the customer who will validate the associated tickets have the information to do so without wasting time?
Issues to validate to close :
-
issue #
Processed issues to keep open or in progress:
-
issue #
Checklist:
-
Does the package check go local? -
Does the CI pass? -
Are the added / fixed features documented, tested? -
Are the added features / solved problems briefly presented in the PR message? -
Are the changes related to tickets / issues that I have listed in the commits and in the PR itself? -
Are the tickets in "review" mode in the Project Tracking Board? -
Does each ticket, if it is to be closed after acceptance of the PR, contain a comment that tells how to validate it?