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: Merge agent and user metadata #204

Merged
merged 3 commits into from
Apr 17, 2024
Merged

fix: Merge agent and user metadata #204

merged 3 commits into from
Apr 17, 2024

Conversation

whiterabbit1983
Copy link
Contributor

@whiterabbit1983 whiterabbit1983 commented Apr 17, 2024

Ellipsis 🚀 This PR description was created by Ellipsis for commit 0ae5d0e.

Summary:

This PR modifies the patch_agent_query and patch_user_query functions to extract, merge, and update metadata for agents and users.

Key points:

  • Added metadata field extraction from update_data in patch_agent_query and patch_user_query functions.
  • Merged new metadata with existing one in the database using concat function.
  • Included updated metadata in the update query for both agents and users.

Generated with ❤️ by ellipsis.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

❌ Changes requested.

  • Reviewed the entire pull request up to 0ae5d0e
  • Looked at 90 lines of code in 2 files
  • Took 1 minute and 42 seconds to review
More info
  • Skipped 0 files when reviewing.
  • Skipped posting 0 additional comments because they didn't meet confidence threshold of 50%.

Workflow ID: wflow_wDGbXZdgbUiFs0gO


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. We'll respond in a few minutes. Learn more here.

@whiterabbit1983 whiterabbit1983 merged commit 5023640 into dev Apr 17, 2024
5 checks passed
@whiterabbit1983 whiterabbit1983 deleted the x/metedata-update branch April 17, 2024 12:02
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