Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automatically generate i18n templates for compendia #602

Draft
wants to merge 57 commits into
base: main
Choose a base branch
from

Conversation

rsek
Copy link
Collaborator

@rsek rsek commented Jan 17, 2023

relies on #601, #596 , #595

WIP -- see the i18n/packs directory. the idea is to provide a friendlier starting point for localization than we currently have. to that end, this PR migrates moves and assets to 100% markdown for their localizable text, and dispenses with move displayNames in favour of using one version of name throughout.

TODO

  • moves
    • migrate localizeable text to markdown
  • assets
    • migrate localizable text to markdown
    • misc bugfix: provide the name property for certain asset abilities (nothing uses it right now, tho)
  • delve site themes + delve site domains
    • do in a nicer key format? something similar to babele's default table range handler would be best, i think, but will require some elbow grease... or a migration of delve features/dangers to a TableResultData-like format
    • merge Migrate delve oracle rows to a TableResult-like format #605 to main + this branch
  • oracles, with babele's default table range format
  • key by ID if there's a clean way to do it
    • worth investigating a way to do it by DFID?
    • is there a converter that we can build into our system?
  • setting truth journal
  • foe Actors
  • foe Items
  • test with Babele
    • do array properties override correctly?
    • what about flags?
  • Update readme with recommend use for localization
  • consider providing this as a cloneable repo for people to get started with?
  • Update CHANGELOG.md

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