Skip to content

fix(chat): Resolve knowledge card display and section regeneration bugs

Edouard Legoupil requested to merge fix-chat-jsx-bugs into main_dev

Created by: Edouard-Legoupil

This commit addresses two distinct issues in the Chat.jsx component and the backend API:

  1. Display All Outcome Knowledge Cards: The logic for fetching and displaying knowledge cards associated with multiple outcomes has been corrected in Chat.jsx. Previously, URLSearchParams was used incorrectly, preventing multiple outcome_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.

  2. Fix Section Regeneration: The section regeneration feature was failing with a 400 Bad Request error because the temporary session created for the regeneration process was missing the proposal_template. The /regenerate_section/{proposal_id} endpoint in backend/api/proposals.py has 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?

Merge request reports

Loading