KEH-739 - Copilot interim #45
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
TotalDwarf03
approved these changes
Mar 19, 2025
Contributor
TotalDwarf03
left a comment
TotalDwarf03
left a comment
There was a problem hiding this comment.
A minor change to make regarding the endpoint URLs. I'm approving in advance so you can crack on tomorrow.
Once the URLs have been sorted, this can be merged 👍
src/pages/interim_page.py
Outdated
Contributor
There was a problem hiding this comment.
Change ONSDigital to use the org variable
rest.get(f"/orgs/{org}/copilot/metrics")
src/pages/interim_page.py
Outdated
src/pages/interim_page.py
Outdated
src/pages/interim_page.py
Outdated
Contributor
There was a problem hiding this comment.
If this wasn't an interim, something like this would need to be in its own function due to repeated logic.
No action needed as this is an interim, just something to be aware of in future
TotalDwarf03
added a commit
that referenced
this pull request
Aug 22, 2025
* Setup interim page for soph * Patch: Modify the dashboard to use example data. * Add interim notice to README * Set code completions and copilot chat metrics with placeholder values * Get usage data from API and initial commit for generating datasets * Group datasets and display metrics on dashboard * Add seat information * Move slider * Use org variable in endpoints --------- Co-authored-by: Kieran Pritchard <kieran.pritchard@ons.gov.uk>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Introduces a new Copilot interim page to display key metrics on IDE code completions and Copilot Chat following updates to the Github API endpoints. The page also displays the seat information which was unaffected by the API change.