Skip to content

Add MinIO connector documentation and update docs.json#127

Open
Kushagra1122 wants to merge 1 commit into
pipeshub-ai:mainfrom
Kushagra1122:minio
Open

Add MinIO connector documentation and update docs.json#127
Kushagra1122 wants to merge 1 commit into
pipeshub-ai:mainfrom
Kushagra1122:minio

Conversation

@Kushagra1122

Copy link
Copy Markdown
Contributor

No description provided.

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request introduces the MinIO connector documentation, including a new MDX guide, the MinIO logo, and an update to the documentation structure in docs.json. The review feedback identifies several instances where UK English spelling was used and recommends switching to US English for terms like 'organizes', 'behavior', and 'organization' to maintain consistency across the documentation site.


PipesHub **only reads** data from MinIO. It does not upload, move, rename, or delete objects unless you use separate tooling for that.

### How MinIO organises data

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.

medium

The documentation uses UK English spelling ("organises"), while other connector documentation (e.g., s3.mdx) primarily uses US English ("organizing"). For consistency across the documentation site, please use US English.

### How MinIO organizes data


### Indexed for text search

Same behaviour as other document connectors — text-heavy formats are indexed for AI search:

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.

medium

The documentation uses UK English spelling ("behaviour"), while other connector documentation (e.g., s3.mdx) uses US English ("behavior"). For consistency across the documentation site, please use US English.

Same behavior as other document connectors — text-heavy formats are indexed for AI search:


- The access key may lack `s3:ListBucket` / list permissions on those buckets.
- Check **Bucket Names** filters — an **exclude** rule can hide everything by mistake.
- Verify you are pointing at the correct MinIO tenant / alias if your organisation runs several clusters.

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.

medium

The documentation uses UK English spelling ("organisation"), while other connector documentation (e.g., s3.mdx) uses US English ("organization"). For consistency across the documentation site, please use US English.

- Verify you are pointing at the correct MinIO tenant / alias if your organization runs several clusters.

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