Skip to content

Hide sap.businessone connector from the store - #47

Merged
daneshk merged 1 commit into
wso2:mainfrom
RDPerera:hide-sap-businessone-connector
Jul 9, 2026
Merged

Hide sap.businessone connector from the store#47
daneshk merged 1 commit into
wso2:mainfrom
RDPerera:hide-sap-businessone-connector

Conversation

@RDPerera

@RDPerera RDPerera commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds sap.businessone to HIDDEN_PACKAGES in src/lib/connector-utils/connector-utils.ts, which excludes it from search results, pagination, and filter options across the connector store.

Description

The sap.businessone is an HTTP wrapper that handles the auth logic of SAP Business One and has nothing to do with being a connector. Therefore, this PR is raised to hide it from the connector store.

Screenshot 2026-07-09 at 10 54 12

Adds sap.businessone to HIDDEN_PACKAGES so it is excluded from
search results, pagination, and filter options.
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This change updates the HIDDEN_PACKAGES constant in connector-utils.ts to include the sap.businessone package, adding it to the set of connector packages excluded from visibility and filtering in the connector store.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive Pre-merge checks only support a single pull request description template, so this description cannot be fully evaluated. Please use the supported single pull request description template so the check can be evaluated consistently.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the main change of hiding sap.businessone from the connector store.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@daneshk
daneshk merged commit 30cf443 into wso2:main Jul 9, 2026
8 checks passed
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