Skip to content

increment project query count for manager features POC #78

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

Merged
merged 6 commits into from
Apr 23, 2025

Conversation

kelsey-wong
Copy link
Contributor

Key Info

  • PRD: link
  • Priority: urgent (want to merge ASAP so we can test for demo tomorrow am)

What changed?

This is a temporary solution to record the total number of "queries" that a RAG app handles when integrating with Codex. In the future, we will not need to make this additional API call because we'll keep records in the planned query_logs Postgres table.

What do you want the reviewer(s) to focus on?

n/a


Checklist

  • Did you link the GitHub issue?
  • Did you follow deployment steps or bump the version if needed?
  • Did you add/update tests?
  • What QA did you do?
    • Tested...

@kelsey-wong kelsey-wong requested a review from axl1313 April 23, 2025 17:35
Copy link
Collaborator

@axl1313 axl1313 left a comment

Choose a reason for hiding this comment

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

lgtm!

@kelsey-wong kelsey-wong merged commit 13686e9 into main Apr 23, 2025
11 checks passed
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.

2 participants