-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add endpoint to get most recently elected boards #599
base: stable
Are you sure you want to change the base?
Conversation
Mentioning it here, but also holds for the other PRs (and will include it on mine in the coming days); would be good if we add some tests. I want to rewrite our testing framework in relation to the mocking of core, but in the meantime we should still add tests |
This is the actual endpoint that I needed as for the AC check, and not the one from the previous commit, but we still might want to use that one for something
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## stable #599 +/- ##
==========================================
+ Coverage 98.26% 98.31% +0.04%
==========================================
Files 13 13
Lines 288 296 +8
Branches 47 49 +2
==========================================
+ Hits 283 291 +8
Misses 5 5 ☔ View full report in Codecov by Sentry. |
Also changed an endpoint to make it a more logical name
Needed for functioning of frontend#2036