Skip to content

feat(ui): support regionless OCI credentials#11742

Open
HugoPBrito wants to merge 3 commits into
feat/oci-regionless-api-chain-11565from
feat/oci-regionless-ui-chain-11565
Open

feat(ui): support regionless OCI credentials#11742
HugoPBrito wants to merge 3 commits into
feat/oci-regionless-api-chain-11565from
feat/oci-regionless-ui-chain-11565

Conversation

@HugoPBrito

@HugoPBrito HugoPBrito commented Jun 30, 2026

Copy link
Copy Markdown
Member

Context

Since OCI credential setup no longer requires a region, the UI should stop asking users for one during provider onboarding.

Description

This PR removes the OCI region field from the credentials form, updates the credential payload builder and related types, and adjusts UI/unit/E2E coverage for regionless OCI setup.

Changed files:

File Change
ui/CHANGELOG.md Adds the UI changelog entry.
ui/components/providers/workflow/forms/via-credentials/oraclecloud-credentials-form.tsx Removes the OCI region input from the credentials form.
ui/hooks/use-credentials-form.ts Updates credential form behavior.
ui/lib/provider-credentials/build-credentials.test.ts Adds UI payload-builder coverage.
ui/lib/provider-credentials/build-credentials.ts Builds regionless OCI credential payloads.
ui/tests/providers/providers-page.ts Updates provider E2E helper behavior.
ui/tests/providers/providers.spec.ts Updates provider E2E coverage.
ui/types/components.ts Removes obsolete region typing.
ui/types/formSchemas.ts Removes OCI region schema requirements.

Steps to review

  1. Review the UI diff against the API slice.
  2. Confirm the UI changes match the UI portion of feat: support regionless OCI provider setup #11565.
  3. Confirm the final chain tree is equivalent to feat: support regionless OCI provider setup #11565.

Validation previously run for this slice:

pnpm vitest run --project unit lib/provider-credentials/build-credentials.test.ts

Checklist

Community Checklist
  • This feature/issue is listed in here or roadmap.prowler.com
  • Is it assigned to me, if not, request it via the issue/feature in here or Prowler Community Slack

SDK/CLI

  • Are there new checks included in this PR? No
    • If so, do we need to update permissions for the provider? Please review this carefully.

UI

  • All issue/task requirements work as expected on the UI
  • If this PR adds or updates npm dependencies, include package-health evidence (maintenance, popularity, known vulnerabilities, license, release age) and explain why existing/native alternatives are insufficient.
  • Screenshots/Video of the functionality flow (if applicable) - Mobile (X < 640px)
  • Screenshots/Video of the functionality flow (if applicable) - Table (640px > X < 1024px)
  • Screenshots/Video of the functionality flow (if applicable) - Desktop (X > 1024px)
  • Ensure new entries are added to CHANGELOG.md, if applicable.

API

  • All issue/task requirements work as expected on the API
  • Endpoint response output (if applicable)
  • EXPLAIN ANALYZE output for new/modified queries or indexes (if applicable)
  • Performance test results (if applicable)
  • Any other relevant evidence of the implementation (if applicable)
  • Verify if API specs need to be regenerated.
  • Check if version updates are required (e.g., specs, uv, etc.).
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Chain Context

Field Value
Chain OCI regionless provider setup split from #11565
Tracker PR #11565
Position 4 of 4
Base feat/oci-regionless-api-chain-11565
Depends on #11741
Follow-up None
Review budget 82 / 400
Starts at API regionless OCI credential support
Ends with UI regionless OCI credential setup; final chain tree matches #11565

Chain Overview

master
 └── #11739 Platform CI guard
      └── #11740 SDK regionless OCI setup
           └── #11741 API regionless OCI credentials
                └── 📍 #11742 UI regionless OCI credentials

Scope

Autonomy

  • CI is expected to pass for this PR branch
  • This PR has one deliverable scope
  • This PR can be rolled back without unrelated changes
  • Tests, docs, or manual verification cover this unit

@HugoPBrito HugoPBrito requested a review from a team as a code owner June 30, 2026 14:12
@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 130af7fe-33a5-4218-9deb-f4948c6c95ae

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/oci-regionless-ui-chain-11565

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pfe-nazaries pfe-nazaries left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@HugoPBrito HugoPBrito added the no-merge Please, DO NOT MERGE this PR. label Jul 8, 2026
@HugoPBrito

Copy link
Copy Markdown
Member Author

Note: Fragment changelog must be used here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-merge Please, DO NOT MERGE this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants