Skip to content

chore: update schema excel template with the missing field types - #6908

Draft
ihar-tsykala wants to merge 2 commits into
developfrom
feat/update-the-excel-template-with-new-properties
Draft

ihar-tsykala wants to merge 2 commits into
developfrom
feat/update-the-excel-template-with-new-properties

Conversation

@ihar-tsykala

Copy link
Copy Markdown
Contributor

Description:

Fixes #

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@ihar-tsykala ihar-tsykala self-assigned this Sep 11, 2026
@ihar-tsykala
ihar-tsykala requested review from a team as code owners September 11, 2026 21:58
@ihar-tsykala
ihar-tsykala marked this pull request as draft September 11, 2026 21:58
@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown

Test Results

   13 files  1 799 suites   2m 18s ⏱️
9 025 tests 9 025 ✅ 0 💤 0 ❌
9 059 runs  9 059 ✅ 0 💤 0 ❌

Results for commit c640df6.

♻️ This comment has been updated with latest results.

@swidnikk

Copy link
Copy Markdown
Contributor

Couple things I've noticed

Enum worksheet: remove the Schema Name Column

The Enums sheet's "Schema name" column is dead weight — readSharedEnumSheet/resolveSharedEnumColumns in common/src/xlsx/xlsx-to-json.ts match headers by name (Enum Name, Loaded to IPFS, Value) and never read a "Schema name" column; enums are matched globally by name, not scoped per schema. Keeping it in the template implies authors should fill it in, but it's just unused overhead. Suggest dropping it from the template/README so the example matches what the parser actually consumes: Enum Name | Loaded to IPFS | Value.

Update the guidance for Enum's

In the Enums tab, the three columns are: Enum Name | Loaded to IPFS | Value

For the first value of each group: fill in Enum Name, Loaded to IPFS. Leave those three columns blank on subsequent value rows for the same group.

Dropdown list — values defined in the Enums tab. The field's Parameter must exactly match the "Enum Name" column.

Parameter column: Type-dependent: enum name (Enum)

image

Description Examples

Suggest to update all description fields to represent what might be presented to the user on a form such as the following,
image

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