Skip to content

docs(ui-accuracy): remove non-existent Build Service overflow-menu step#1281

Merged
codyjlandstrom merged 2 commits into
mainfrom
docs/ui-accuracy/align-admin-ui-labels
Jul 2, 2026
Merged

docs(ui-accuracy): remove non-existent Build Service overflow-menu step#1281
codyjlandstrom merged 2 commits into
mainfrom
docs/ui-accuracy/align-admin-ui-labels

Conversation

@ekline

@ekline ekline Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Context

This PR fixes a factual mismatch between documentation and the Okteto frontend UI, verified against the current component code under app/frontend/src. Cosmetic and stylistic changes are intentionally out of scope.

Fix

File Location Doc said Frontend shows Fix
admin/build-service.mdx How to Restart, step 2 "Click the Restart button (or the overflow menu → Restart)" BuildServiceTable.tsx renders only a direct Restart button; no overflow/more menu exists Removed the non-existent overflow-menu alternative

How to verify

  • build-service.mdx: the Restart button (no overflow menu) in app/frontend/src/features/BuildService/Admin/BuildServiceTable.tsx:164-165; the following step's Delete and recreate confirm label is accurate (BuildServiceTable.tsx:219).

Reverted from this PR

Two earlier UI-accuracy edits were reverted after review — they were determined to be inaccurate:

  • admin/cloud-credentials/gcp.mdx — proposed OIDC endpoint path change (reverted to original).
  • admin/cloud-credentials/aws.mdx — proposed "Audience JWT Claim" label change (reverted to original).

📝 Created with EkLine · View session

Signed-off-by: ekline[bot] <202747777+ekline[bot]@users.noreply.github.com>
@netlify

netlify Bot commented Jun 30, 2026

Copy link
Copy Markdown

Deploy Preview for okteto-docs ready!

Name Link
🔨 Latest commit e2b6b08
🔍 Latest deploy log https://app.netlify.com/projects/okteto-docs/deploys/6a46f985f48a0b0008eb49bb
😎 Deploy Preview https://deploy-preview-1281--okteto-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.


Within the newly created Workload Identity Pool, register your Kubernetes cluster as an OIDC Identity Provider in GCP.
To do this, Okteto provides the OIDC endpoint of your cluster, which can be found under the Integrations -> Cloud Credentials section of your Okteto Admin Dashboard.
To do this, you can find the OIDC endpoint of your cluster under the Integrations -> General section of your Okteto Admin Dashboard.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UI accuracy (High): the doc previously sent readers to Integrations → Cloud Credentials for the OIDC endpoint. The OIDC issuer and subject are actually rendered on the General Integrations page — OidcIntegration is mounted inside AdminIntegrations (admin integrations route), confirmed in app/frontend/src/features/AdminIntegrations/AdminIntegrations.tsx:63 and OidcIntegration.tsx. The Cloud Credentials page has no OIDC display. Path corrected to Integrations → General. The sibling amazon-ecr.mdx already describes it this way.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted — this UI-accuracy edit was determined to be inaccurate and the file has been restored to its original content. Disregard this suggestion.

- **Role ARN**: The Role ARN `ROLE_ARN` you created in Step 2.
- **Region**: The AWS Region for the AWS STS Regional Endpoint. You can find more information about regional endpoints [here](https://docs.aws.amazon.com/IAM/latest/UserGuide/id_credentials_temp_enable-regions.html) and [here](https://docs.aws.amazon.com/sdkref/latest/guide/feature-sts-regionalized-endpoints.html)
- **Audience**: The Audience `AUDIENCE` that we defined in Step 1.
- **Audience JWT Claim**: The Audience `AUDIENCE` that we defined in Step 1.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UI accuracy (Medium): the AWS IAM Role form field is labeled Audience JWT Claim, not Audience. Confirmed in app/frontend/src/features/CloudCredentialsAdmin/ChangeModal/AwsForm.tsx:44 (required('Audience JWT Claim')) and :116 (FormField.Label text "Audience JWT Claim"). The label now matches the form, and is consistent with the sibling gcp.mdx which already uses "Audience JWT Claim".

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reverted — this UI-accuracy edit was determined to be inaccurate and the file has been restored to its original content. Disregard this suggestion.


1. Locate the pod you want to restart in the Build Service dashboard
2. Click the **Restart** button (or the overflow menu → **Restart**)
2. Click the **Restart** button

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UI accuracy (Medium): removed the (or the overflow menu → Restart) alternative — no overflow/more menu exists in the Build Service pod table. The table renders only a direct Restart button, confirmed in app/frontend/src/features/BuildService/Admin/BuildServiceTable.tsx:164-165; a grep for overflow/more-menu components in that file returns nothing. The following step's Delete and recreate confirm label is also accurate (BuildServiceTable.tsx:219).

Signed-off-by: ekline[bot] <202747777+ekline[bot]@users.noreply.github.com>
@ekline ekline Bot changed the title docs(ui-accuracy): align admin UI labels with current frontend docs(ui-accuracy): remove non-existent Build Service overflow-menu step Jul 2, 2026
@codyjlandstrom codyjlandstrom merged commit 1d5755e into main Jul 2, 2026
4 checks passed
@codyjlandstrom codyjlandstrom deleted the docs/ui-accuracy/align-admin-ui-labels branch July 2, 2026 23:57
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.

1 participant