Skip to content

Dashboard: optimize for .com #12389

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 16 commits into from
Aug 11, 2025
Merged

Dashboard: optimize for .com #12389

merged 16 commits into from
Aug 11, 2025

Conversation

@stsewd
Copy link
Member

@stsewd stsewd commented Aug 7, 2025 •
edited
Loading

  • Use an internal cache for checking if an org has sso enabled. While we cache the results in redis, this still requires checking redis, and then hitting the DB, a local cache avoids that.
  • Instead of doing two queries for getting the teams (admin and read only), just do it in one.

@stsewd stsewd marked this pull request as ready for review August 11, 2025 18:48
@stsewd stsewd requested a review from a team as a code owner August 11, 2025 18:48
@stsewd stsewd requested a review from agjohnson August 11, 2025 18:48
@stsewd stsewd requested a review from ericholscher August 11, 2025 18:58
@stsewd stsewd merged commit ac69118 into main Aug 11, 2025
7 checks passed
@stsewd stsewd deleted the optimize-dashboard-dom branch August 11, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericholscher ericholscher ericholscher approved these changes

@agjohnson agjohnson Awaiting requested review from agjohnson agjohnson is a code owner automatically assigned from readthedocs/backend

Assignees

@stsewd stsewd

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stsewd @ericholscher