Skip to content

reproduce a bug with a complex calculation #593

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 2 commits into from
Jul 15, 2025

Conversation

jsw800
Copy link
Contributor

@jsw800 jsw800 commented Jul 14, 2025

Contributor checklist

This is a reproduction of the issue I brought up here: https://discord.com/channels/711271361523351632/1394442077420650626/1394442077420650626

It seems like something could be confusing the names of the level field on Folder and the level field on FolderItem, although I have yet to find a root cause.

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

@jsw800
Copy link
Contributor Author

jsw800 commented Jul 14, 2025

I actually didn't quite get this running as-is on my machine since I ran into some issues with any_value vs array_agg in sql_implementation.ex, seemingly due to my local postgres version, but I did a smoke test by changing any_value to array_agg in there and I did see the same error I was seeing in my real project. I assume I just have something configured wrong for running tests locally

@jsw800
Copy link
Contributor Author

jsw800 commented Jul 14, 2025

Can confirm the test fails as expected in CI, must be just an issue with my local configuration

@zachdaniel zachdaniel merged commit 8787615 into ash-project:main Jul 15, 2025
61 of 64 checks passed
@zachdaniel
Copy link
Contributor

🚀 Thank you for your contribution! 🚀

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