Skip to content

feat: import Zimbra Drive app and ZeXtras publisher page#99

Open
DeepDiver1975 wants to merge 1 commit into
mainfrom
import-zextras
Open

feat: import Zimbra Drive app and ZeXtras publisher page#99
DeepDiver1975 wants to merge 1 commit into
mainfrom
import-zextras

Conversation

@DeepDiver1975

Copy link
Copy Markdown
Member

Imports the Zimbra Drive app (v0.8.25) from the legacy marketplace (marketplace.owncloud.com) together with its publisher ZeXtras.

Details

  • Validates against the same checks CI runs (validateRelease); the two screenshots are fetched from the original listing and committed locally as same-origin assets (Git LFS).
  • publishers/zextras/publisher.jsonenabled: false (page was private, isPagePublic: false), claiming zimbradrive.
  • Historical downloads (7,082) and release date (2020-04-02) recorded in the data files.

Supersedes

Replaces the placeholder publisher PR #91, which will be closed.

🤖 Generated with Claude Code

Imports the Zimbra Drive app (v0.8.25) from the legacy marketplace
(marketplace.owncloud.com) and adds its publisher, ZeXtras.

Validates against the same checks CI runs (validateRelease); the two
screenshots are fetched from the original listing and committed locally
as same-origin assets. Historical download total (7,082) and the real
release date (2020-04-02) are recorded in data/downloads-baseline.json
and data/created.json.

Publisher zextras is disabled (enabled: false) — its page was private on
the old marketplace (isPagePublic: false) — and claims zimbradrive.
Supersedes the placeholder publisher PR #91.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
@DeepDiver1975

Copy link
Copy Markdown
Member Author

Review: import Zimbra Drive app + ZeXtras publisher — looks good.

  • Schema/structure: publishers/zextras/publisher.json matches the standard publisher shape (slug, name, enabled, website, description, apps) used across all other publishers. The app release layout (apps/zimbradrive/releases/0.8.25/{package.tar.gz,screenshots/}) mirrors existing apps (e.g. wallpaper); no per-app JSON metadata is expected (it lives in the package). data/created.json and data/downloads-baseline.json entries are well-formed.
  • LFS: all three binaries (package.tar.gz, 01.png, 02.png) are committed as valid Git LFS pointers, matching the .gitattributes rules for apps/**/*.{tar.gz,png}.
  • Ownership integrity: only zextras claims zimbradrive — no duplicate or conflicting publisher. The placeholder publisher PR feat: add ZeXtras publisher page #91 it supersedes is already CLOSED, so there is no stale leftover.
  • Privacy: enabled: false correctly reflects the original private (isPagePublic: false) listing.
  • Secrets/links: no secrets; website link is the public company page.
  • CI: validate is green.

No changes requested.

🤖 Generated with Claude Code

1 similar comment
@DeepDiver1975

Copy link
Copy Markdown
Member Author

Review: import Zimbra Drive app + ZeXtras publisher — looks good.

  • Schema/structure: publishers/zextras/publisher.json matches the standard publisher shape (slug, name, enabled, website, description, apps) used across all other publishers. The app release layout (apps/zimbradrive/releases/0.8.25/{package.tar.gz,screenshots/}) mirrors existing apps (e.g. wallpaper); no per-app JSON metadata is expected (it lives in the package). data/created.json and data/downloads-baseline.json entries are well-formed.
  • LFS: all three binaries (package.tar.gz, 01.png, 02.png) are committed as valid Git LFS pointers, matching the .gitattributes rules for apps/**/*.{tar.gz,png}.
  • Ownership integrity: only zextras claims zimbradrive — no duplicate or conflicting publisher. The placeholder publisher PR feat: add ZeXtras publisher page #91 it supersedes is already CLOSED, so there is no stale leftover.
  • Privacy: enabled: false correctly reflects the original private (isPagePublic: false) listing.
  • Secrets/links: no secrets; website link is the public company page.
  • CI: validate is green.

No changes requested.

🤖 Generated with Claude Code

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