Skip to content

Conversation

@braddf
Copy link
Contributor

@braddf braddf commented Jul 15, 2025

Pull Request

Description

  • Rename due to NG buying/taking over WPD's DNOs.
  • Reset aggregation to GSP when switching to Delta view

Fixes #620
Fixes #622

How Has This Been Tested?

image

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@braddf braddf requested a review from peterdudfield July 15, 2025 08:49
@braddf braddf self-assigned this Jul 15, 2025
@braddf braddf added the data label Jul 15, 2025
@vercel
Copy link

vercel bot commented Jul 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nowcasting-app ✅ Ready (Inspect) Visit Preview 23 resolved Jul 15, 2025 9:57am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
quartz-app ⬜️ Ignored (Inspect) Visit Preview Jul 15, 2025 9:57am

@peterdudfield
Copy link
Contributor

Ive approved, but ive seen you marked this as a draft. Let me know if you add any more stuff, and I can have another look

@braddf
Copy link
Contributor Author

braddf commented Jul 15, 2025

Cool yep thanks @peterdudfield, was just the other PR that you've just approved, so should be good 👌

@braddf braddf marked this pull request as ready for review July 15, 2025 09:55
@braddf braddf merged commit 47b2d53 into staging Jul 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants