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

Fix retrieving a conversation tracker from HTTP API endpoint dependent on query params (3.3.x) #12192

Merged
merged 2 commits into from
Mar 28, 2023

Conversation

ancalita
Copy link
Member

@ancalita ancalita commented Mar 28, 2023

(cherry picked from commit e895752)

Proposed changes:

  • Backported merged bugfix from 3.4.x

Status (please check what you already did):

  • added some tests for the functionality
  • updated the documentation
  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

…t on query params (#12139)

* implement fix

* update fix

* add unit tests for tracker store update

* add tests for processor and server updates

* implement fix for story endpoint bug, add tests, implement retrieve_full_tracker for InMemoryTrackerStore

* fix failing tests

* add retrieve_full_tracker implementation to redis and dynamodb + tests for InMemoryTS

* simplify tracker store new tests, add tests for mongodb

* fix redis tracker saving mechanism, add tests

* correct dynamodb update to retrieve method

* address review feedback

* add integration testing for postgres, redis, mongodb

* add changelog entry

(cherry picked from commit e895752)
@github-actions
Copy link
Contributor

🚀 A preview of the docs have been deployed at the following URL: https://12192--rasahq-docs-rasa-v2.netlify.app/docs/rasa

@ancalita ancalita marked this pull request as ready for review March 28, 2023 15:02
@ancalita ancalita requested review from a team as code owners March 28, 2023 15:02
@ancalita ancalita requested review from Tawakalt, rasa-aadlv and radovanZRasa and removed request for a team March 28, 2023 15:02
@ancalita ancalita merged commit 874dc2d into 3.3.x Mar 28, 2023
@ancalita ancalita deleted the ATO-694-backport-bugfix-3.3.x branch March 28, 2023 15:31
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.

3 participants