Skip to content

Conversation

@graysonguarino
Copy link
Collaborator

Development still in progress. Sharing just for visibility.

Issue Number:
Reference the issue this PR fixes.

N/A

Description of Changes:
Provide a clear and concise explanation of what changes you made and why.

Testing Done:
How did you test your changes? Share details like steps, tools used, or results.

Terms of contribution:

By submitting this pull request, I agree that this contribution is licensed under the terms of the Apache License, Version 2.0.


Thanks for submitting your pull request! We will review it as soon as possible.

@hallyn-geico
Copy link
Collaborator

All looks good to me.

@graysonguarino
Copy link
Collaborator Author

graysonguarino commented May 19, 2025

All looks good to me.

Talked with @ecpullen about this PR and he thinks the queries get a bit complex and nested, thus likely a bit difficult to maintain (and I agree). Currently putting the finishing touches on some changes to the API that make the data flatter (#26). Will close this PR once I get the other one finished, but was temporarily leaving this up for comparison.

@graysonguarino
Copy link
Collaborator Author

Closing in favor of #26

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