Skip to content

feat: rebrand app metadata to Pinner S3 Server - #57

Merged
pcfreak30 merged 1 commit into
developfrom
feat/app-metadata-pinner-s3
Jul 21, 2026
Merged

feat: rebrand app metadata to Pinner S3 Server#57
pcfreak30 merged 1 commit into
developfrom
feat/app-metadata-pinner-s3

Conversation

@pcfreak30

@pcfreak30 pcfreak30 commented Jul 21, 2026

Copy link
Copy Markdown
Member

Update Sia app metadata to reflect the Pinner S3 Server branding and LumeWeb GitHub org. The `s3d` app ID is intentionally preserved to avoid breaking existing registrations.

Changes:

  • `internal/backend/sia.go` — new `SiaAppMetadata()` helper, canonical source for `sdk.AppMetadata`
  • `internal/backend/factory.go` — replaces inline struct with `SiaAppMetadata()`
  • `internal/backend/manager.go` — same, unifies description with the canonical one
  • `internal/onboarding/onboarding.go` — maps `SiaAppMetadata()` fields into `ConfigResponse`

This pull request rebrands the application's metadata to Pinner S3 Server and centralizes how that metadata is defined and consumed across the service.

Changes:

  • Introduces a single canonical source, SiaAppMetadata(), that returns the app's registration metadata for the Sia storage SDK.
  • Updates the app name from "S3d" / "s3-server-panel" to "Pinner S3 Server".
  • Updates the service URL from the Siafoundation/s3d repository to the LumeWeb/s3-server repository.
  • Replaces hard-coded metadata in the backend factory, account client, and onboarding config endpoint with calls to the shared SiaAppMetadata() function.

Functional impact:

  • Ensures consistent app identity (name, description, logo, and service URL) everywhere the application registers itself with the Sia SDK or exposes its configuration to clients.
  • Eliminates drift between the backend's SDK registration and the onboarding/config endpoint.

@kody-ai

This comment has been minimized.

Comment thread internal/onboarding/onboarding.go
@pcfreak30
pcfreak30 force-pushed the feat/app-metadata-pinner-s3 branch from 5e25bd2 to c70436f Compare July 21, 2026 09:09
@kody-ai

This comment has been minimized.

@pcfreak30
pcfreak30 force-pushed the feat/app-metadata-pinner-s3 branch from c70436f to ae3b824 Compare July 21, 2026 09:10
- Rename app name from "S3d" / "s3-server-panel" to "Pinner S3 Server"
- Update GitHub URL from SiaFoundation/s3d to LumeWeb/s3-server
- Introduce backend.SiaAppMetadata() as single source of truth
- Consume it in factory.go, manager.go, and onboarding.go
- AppID left unchanged (types.HashBytes([]byte("s3d")))
@pcfreak30
pcfreak30 force-pushed the feat/app-metadata-pinner-s3 branch from ae3b824 to 456dea8 Compare July 21, 2026 09:13
@kody-ai

kody-ai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Kody Review Complete

Great news! 🎉
No issues were found that match your current review configurations.

Keep up the excellent work! 🚀

Kody Guide: Usage and Configuration
Interacting with Kody
  • Request a Review: Ask Kody to review your PR manually by adding a comment with the @kody start-review command at the root of your PR.

  • Validate Business Logic: Ask Kody to validate your code against business rules by adding a comment with the @kody -v business-logic command.

  • Provide Feedback: Help Kody learn and improve by reacting to its comments with a 👍 for helpful suggestions or a 👎 if improvements are needed.

Current Kody Configuration
Review Options

The following review options are enabled or disabled:

Options Enabled
Bug
Performance
Security
Business Logic

Access your configuration settings here.

@pcfreak30
pcfreak30 merged commit 6935f76 into develop Jul 21, 2026
11 checks passed
@github-actions

Copy link
Copy Markdown

Docker image for testing

Pull the image for this PR:

docker pull ghcr.io/lumeweb/s3-server:sha-25b114f

Or use in docker-compose:

services:
  s3-server:
    image: ghcr.io/lumeweb/s3-server:sha-25b114f

The image is rebuilt on every push to this PR's branch.

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