Skip to content

Catalog JadeRealms Moon Gate as a publisher-submitted source for review - #126

Open
hudijiang wants to merge 1 commit into
TMHSDigital:mainfrom
hudijiang:codex/add-jaderealms-moon-gate
Open

hudijiang wants to merge 1 commit into
TMHSDigital:mainfrom
hudijiang:codex/add-jaderealms-moon-gate

Conversation

@hudijiang

Copy link
Copy Markdown

Adds one free downloadable asset, JadeRealms Moon Gate, to the 3D catalog. This follows the pack-level scope rather than adding a publisher-wide card. The GLB and complete model ZIP containing Blender sources are accessible from https://3dmodelhub.io/models/moon_gate/.

License handling:

  • Custom license; commercial game and animation use is explicitly permitted by https://3dmodelhub.io/license/. Standalone resale and repackaged redistribution are prohibited.
  • Kept at needs-review as required for publisher-submitted entries. Attribution remains unknown because the live public terms do not state a credit requirement or waiver. Third-party material rights and provenance need independent maintainer review; no blanket originality or CC0 claim is made.
  • The entry covers only the free individual model. Paid arranged scene kits advertised alongside it are identified in Notes.

The live model and license pages were read on 2026-10-22. Evidence includes dated source links and quotations. No files are rehosted. Updated the category index and used the count-sync script for all restated counts.

Disclosure: submitted on behalf of JadeRealms as our own resource. Prepared with AI assistance. This is a new-source submission; no existing issue was found for JadeRealms or 3dmodelhub.io.

Validation passed: npm test (134 catalog-check assertions and 404 library assertions), npm run validate (416 entries), npm run build (416 entry pages), and git diff --check.

@TMHSDigital

Copy link
Copy Markdown
Owner

Thanks for the submission and the clear disclosure. I re-read the live model and licence pages and the licence text inside the complete ZIP; they agree with the Evidence quotes, and the downloads work without sign-in.

One thing blocks the merge: #124 landed this morning and made added a required field, so the validator now fails on this entry:

catalog/3d/jaderealms-moon-gate.md missing/empty: added

Please add this line to the frontmatter, before verified:, and push:

added: 2026-10-22

The entry otherwise merges cleanly on current main. It stays at needs-review until a maintainer re-read, as CONTRIBUTING describes for publisher-submitted entries.

@TMHSDigital

Copy link
Copy Markdown
Owner

One more thing since my last comment: #129 merged this morning and main is now at 416 entries, so this branch conflicts on the restated counts. When you add the added: line, please also rebase onto current main and re-run node site/sync-counts.mjs. The expected results are 417 total and 3D at 78 in README.md, catalog/README.md and site/config.json (expectedEntryCount). Everything else in the entry is fine as it stands.

@TMHSDigital

Copy link
Copy Markdown
Owner

Following up on my earlier two comments, which are still open (the added: 2026-10-22 line and a rebase with node site/sync-counts.mjs). I re-read the live pages today (2026-10-29). The Evidence quotes still match, but the licence page has changed in two ways that the entry needs to reflect:

  1. AI-assisted creation. The licence page now says: "Our assets use an AI-assisted creation process." Per docs/ai-assets.md, please add ai-generated to tags and a Notes line saying so, with a dated Evidence line quoting it.
  2. Scene kits. Notes says paid scene kits are advertised separately, but the site now says "Models and all currently published scene packs are free to download, without sign-in or payment." Please update or drop that Notes line.

(The CC BY 4.0 text on that page covers the separate "Scenario" collection, not this model, so no change is needed there.)

With all four changes and a rebase onto current main, this is ready to merge at needs-review.

This branch has not been deployed

No deployments
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