Skip to content

ToJ set updates + artist credit fixes#34

Open
kayorga wants to merge 22 commits intothroneteki-playtesting:developmentfrom
kayorga:draft
Open

ToJ set updates + artist credit fixes#34
kayorga wants to merge 22 commits intothroneteki-playtesting:developmentfrom
kayorga:draft

Conversation

@kayorga
Copy link
Copy Markdown

@kayorga kayorga commented Aug 13, 2025

No description provided.

Comment thread packs/ToJ.json Outdated
Copy link
Copy Markdown
Collaborator

@stopfstedt stopfstedt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i'm ready from my end to go live with this, i rolled the code-changes to support this set out to TDB just now.

the set needs a publication date and the workInProgress flag toggled to false, otherwise these card will show up as "work in progress" on the site.

some of the cards, like the one below, are still using the placeholder/auto-generated images.

image

also, that draftmancer/ToJ.txt file - is this needed in this repo? if not then please delete.

Copy link
Copy Markdown
Collaborator

@stopfstedt stopfstedt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comments above.

@stopfstedt stopfstedt self-requested a review November 27, 2025 04:45
@kayorga kayorga marked this pull request as ready for review March 2, 2026 23:31
Copy link
Copy Markdown
Collaborator

@Patane97 Patane97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes seem good from my perspective with TheIronThrone.

@kayorga Just mentioning where this should be merged into; as the intent is for us to bring this to the live site (ie. workInProgress: false), I would recommend this actually merge into throneteki:master, rather than playtesting.

The changes from there will then (eventually) flow into development; I will handle the merge conflicts (that is an issue I have accepted with the playtesting -> live workflow for a while now, this would be no exception).

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.

3 participants