fix: Resolve knowledge card display and section regeneration bugs
Created by: Edouard-Legoupil
This commit addresses two distinct issues in the application:
-
Display All Outcome Knowledge Cards: The backend API has been updated to correctly handle requests for multiple outcome knowledge cards. The
get_knowledge_cardsendpoint inbackend/api/knowledge.pynow accepts a list ofoutcome_ids and uses anANYclause in the database query to fetch all matching cards. The corresponding frontend code inChat.jsxhas been reverted to use the standardURLSearchParamsAPI. -
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.
Additionally, this commit includes fixes for several pre-existing test failures that were discovered during the development process.
- 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?