Skip to content

fix/users: remove query for usageStatistics#1264

Merged
keegancsmith merged 3 commits intomainfrom
k/pr-fix-users-usage-statistics
Mar 2, 2026
Merged

fix/users: remove query for usageStatistics#1264
keegancsmith merged 3 commits intomainfrom
k/pr-fix-users-usage-statistics

Conversation

@keegancsmith
Copy link
Member

@keegancsmith keegancsmith commented Mar 2, 2026

This field was deprecated in March last year and was removed November
last year 1. Before this change any user subcommand just failed. Will be
adding some integration smoke tests to catch these sort of issues.

Additionally found one place we didn't set Content-Type which
Sourcegraph also recently started enforcing.

This field was deprecated in March last year and was removed November
last year [1]. Before this change any user subcommand just failed. Will be
adding some integration smoke tests to catch these sort of issues.

Additionally found one place we didn't set Content-Type which
Sourcegraph also recently started enforcing.

[1]: https://sourcegraph.sourcegraph.com/deepsearch/3e8160f0-e54a-4b0b-b65a-7bd0b78120ae
@keegancsmith keegancsmith requested review from a team and eseliger March 2, 2026 10:55
@keegancsmith keegancsmith marked this pull request as ready for review March 2, 2026 10:55
@keegancsmith keegancsmith enabled auto-merge (squash) March 2, 2026 11:38
@keegancsmith keegancsmith merged commit fcfad36 into main Mar 2, 2026
8 checks passed
@keegancsmith keegancsmith deleted the k/pr-fix-users-usage-statistics branch March 2, 2026 11:51
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.

3 participants