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

Implement IDs for Chains in GQL #1370

Closed
2 tasks done
JamesonRGrieve opened this issue Jan 18, 2025 · 0 comments · Fixed by #1376
Closed
2 tasks done

Implement IDs for Chains in GQL #1370

JamesonRGrieve opened this issue Jan 18, 2025 · 0 comments · Fixed by #1376
Assignees
@Josh-XT
Labels
area | back end Improvements or additions to the API server or runtime. type | report | bug Confirmed bug in source code.

Comments

@JamesonRGrieve
Copy link

JamesonRGrieve commented Jan 18, 2025 •
edited
Loading

Bug Description

Chains currently pass None back as their ID, using UUIDs will allow path-based indexing on the front end instead of query params, and without having to make names URL-safe. They are present on the REST endpoint.

Image

Proposed Solution

Implement UUID IDs for Chains in GQL.

Acknowledgements

  • I have searched the existing issues to make sure this feature has not been requested yet.
  • I have provided enough information for everyone to understand why this feature request is needed in AGiXT.
@JamesonRGrieve JamesonRGrieve added area | back end Improvements or additions to the API server or runtime. type | request | enhancement Expansion upon an existing feature. labels Jan 18, 2025
@JamesonRGrieve JamesonRGrieve changed the title Implement IDs for Chains Implement IDs for Chains in GQL Jan 18, 2025
@JamesonRGrieve JamesonRGrieve added type | report | bug Confirmed bug in source code. and removed type | request | enhancement Expansion upon an existing feature. labels Jan 18, 2025
Josh-XT added a commit that referenced this issue Jan 19, 2025
@Josh-XT
Fix #1370
2aad9c2
@Josh-XT Josh-XT mentioned this issue Jan 19, 2025
Merged
Josh-XT added a commit that referenced this issue Jan 19, 2025
@Josh-XT
Fix #1370 (#1376)
89c1edc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Josh-XT Josh-XT

Labels
area | back end Improvements or additions to the API server or runtime. type | report | bug Confirmed bug in source code.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #1370: Implement IDs for Chains in GQL Josh-XT/AGiXT
2 participants
@JamesonRGrieve @Josh-XT