Skip to content

API: return extra fields on build details for superusers#12810

Open
humitos wants to merge 1 commit intomainfrom
humitos/build-detail-for-superuser
Open

API: return extra fields on build details for superusers#12810
humitos wants to merge 1 commit intomainfrom
humitos/build-detail-for-superuser

Conversation

@humitos
Copy link
Member

@humitos humitos commented Feb 24, 2026

This is required to show extra fields in the build details page for superusers, such as the builder used to run the build.

This is required to show extra fields in the build details page for
superusers, such as the builder used to run the build.
@humitos humitos marked this pull request as ready for review February 24, 2026 10:53
@humitos humitos requested a review from a team as a code owner February 24, 2026 10:53
@humitos humitos requested a review from stsewd February 24, 2026 10:53
humitos added a commit to readthedocs/ext-theme that referenced this pull request Feb 24, 2026
This is a feature we had in the previous dashboard,
but we lost it in the migration.

It's pretty useful when debugging support request and I'd like to have it back.

Requires readthedocs/readthedocs.org#12810
humitos added a commit to readthedocs/ext-theme that referenced this pull request Feb 24, 2026
This is a feature we had in the previous dashboard,
but we lost it in the migration.

It's pretty useful when debugging support request and I'd like to have it back.

Requires readthedocs/readthedocs.org#12810
@stsewd
Copy link
Member

stsewd commented Feb 24, 2026

I don't really like that we are introducing checks for users again (we moved away from that for API v2). I also think we were working on replacing this with API v3.

@humitos
Copy link
Member Author

humitos commented Feb 24, 2026

How we can achieve the same goal without checking for superusers here? We want to show extra data only to ourselves in the dashboard.

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.

2 participants