feat(copilot): Adds engagement and seat analysis metrics #3241
Conversation
This file contains 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
Changed Packages
|
5 tasks
04kash
reviewed
Mar 19, 2025
workspaces/copilot/plugins/copilot-backend/migrations/202503111030_add_seat_analysys.js
Outdated
Show resolved
Hide resolved
Fixed them, and also fixed a bug where I fetched the wrong datapoint for some metrics. |
|
Should be good to merge after conflicts are resolved |
Changes the backend to use octokit in order to have pagination Signed-off-by: Henrik Edegård <[email protected]>
Signed-off-by: Henrik Edegård <[email protected]>
Signed-off-by: Henrik Edegård <[email protected]>
Signed-off-by: Henrik Edegård <[email protected]>
Signed-off-by: Henrik Edegård <[email protected]>
Signed-off-by: Henrik Edegård <[email protected]>
Signed-off-by: Henrik Edegård <[email protected]>
henrikedegrd
force-pushed
the
copilot-add-engagement
branch
from
1aec1a2 to
75704af
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Hey, I just made a Pull Request!
Adds metrics for engagement and seat analysis.
It adds a few new metrics:
The new metrics are presented with the value from the last day of the selected period range, since an average of the above does not make sense.
It also show some new metrics from the previous change to the new metrics API:
Average metrics of the above are calculated excluding weekends, since that can make the average a bit missleading.
Backend also changed to use octokit to fetch data from Github.
Also found that when using PostgreSQL as database, the unique index using null are not working as expected. So the new metrics use empty string instead. This will have to be addressed for the other metrics also. Will do that in a separate PR.
✔️ Checklist
Signed-off-byline in the message. (more info)