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.

Sign up for GitHub

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

Jump to bottom

[CAI-155] Chatbot/queries fetching #1131

Merged
merged 31 commits into from
Sep 20, 2024
Merged

[CAI-155] Chatbot/queries fetching #1131

merged 31 commits into from
Sep 20, 2024

Conversation

batdevis
Copy link
Collaborator

List of Changes

fetch the queries by sessionId from dynamodb

Motivation and Context

The frontend needs the list of the queries of a session.

How Has This Been Tested?

With local dynamodb.

@batdevis
feat(chatbot): dynamodb infrastructure
3c7a25f
@batdevis
feat(chatbot): dynamodb data attribute
56917dc
@batdevis
fix(chatbot): use string type for map type on dynamodb terraform module
533cdb1
@batdevis
Merge branch 'main' into chatbot/dynamodb
ecbe664
@batdevis
fix(chatbot): dynamodb terraform
d0e003f
@batdevis
feat(chatbot): dynamodb configuration
0495c4e
@batdevis
feat(chatbot): dynamodb table name
96bab4c
@batdevis
chore(chatbot): docker for local development
aecce22
@batdevis
chore(chatbot): docker for local development
138b0e5
@batdevis
doc(chatbot): index creation readme
235c0af
@batdevis
feat(chatbot): write to dynamodb
dc48c19
@batdevis
feat(chatbot): table name var
a61b9fc
@batdevis
feat(chatbot) query creation on dynamodb
14c1051
@batdevis
Merge branch 'main' into chatbot/query-creation
ac23f65
@batdevis
chore(chatbot): API improvements and refactor)
6070813
@batdevis
feat(chatbot): sessionId key in dynamodb
2edc4d5
@batdevis
feat(chatbot): query by sessionId
7508e48
@batdevis batdevis requested a review from a team as a code owner September 17, 2024 14:34
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Sep 17, 2024 •
edited
Loading

⚠️ No Changeset found

Latest commit: 747062e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

christian-calabrese
christian-calabrese approved these changes Sep 20, 2024
View reviewed changes
Copy link
Contributor

@christian-calabrese christian-calabrese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions GitHub Actions
Copy link
Contributor

Branch is not up to date with base branch

@batdevis it seems this Pull Request is not updated with base branch.
Please proceed with a merge or rebase to solve this.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 20, 2024 •
edited by jira bot
Loading

Jira Pull Request Link

This Pull Request refers to the following Jira issue CAI-155

@batdevis batdevis merged commit ad78ab2 into main Sep 20, 2024
13 checks passed
@batdevis batdevis deleted the chatbot/queries-fetching branch September 20, 2024 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sebastiano-Bertolin Sebastiano-Bertolin Sebastiano-Bertolin approved these changes

@christian-calabrese christian-calabrese christian-calabrese approved these changes

Assignees

@batdevis batdevis

Labels
infra size/small
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@batdevis @christian-calabrese @Sebastiano-Bertolin