Skip to content

Conversation

@birkjernstrom
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Nov 27, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
polar Ignored Ignored Preview Nov 28, 2025 0:08am
polar-sandbox Ignored Ignored Preview Nov 28, 2025 0:08am

@birkjernstrom birkjernstrom force-pushed the fix/customer-meter-query branch from dd644fe to df34ea3 Compare November 27, 2025 17:41
@birkjernstrom birkjernstrom marked this pull request as draft November 27, 2025 17:42
@birkjernstrom
Copy link
Member Author

Haven't been able to test this locally (dev environment is completely broken). But this should theoretically work and at least in draft mode, and serve for collaboration/discussion.

Another challenge: This query gets expanded upon in customer_meter.service.update_customer_meter:137+ and customer_meter.service.update_customer_meter:172+

@birkjernstrom
Copy link
Member Author

Before I forget: In regards to the extra conditionals that are set and modifications to the query... Perhaps they need to be set on the inner queries of the UNION. So solution might have to be a final "wrapper" of the ultimate query that creates a union of it.

@psincraian psincraian marked this pull request as ready for review November 28, 2025 13:08
@psincraian psincraian merged commit e5ad012 into main Nov 28, 2025
11 of 12 checks passed
@psincraian psincraian deleted the fix/customer-meter-query branch November 28, 2025 13:18
psincraian added a commit that referenced this pull request Nov 28, 2025
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.

4 participants