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

PCTB: Add the bonus card #82

Closed
wants to merge 1 commit into from
Closed

PCTB: Add the bonus card #82

wants to merge 1 commit into from

Conversation

kodawah
Copy link
Contributor

@kodawah kodawah commented Oct 19, 2023

qq: should we add the (non-legal) thick display commanders in the sideboard section now?

@taw
Copy link
Owner

taw commented Oct 19, 2023

I think the number is 722a not 772a.

Do we want to list unrelated cards that just happen to come in the same box? It feels like a different use case than display commanders. @axxroytovu Any opinion?

@axxroytovu
Copy link
Contributor

It would be consistent with how we track the bonus cards for other secret lair products.

@kodawah
Copy link
Contributor Author

kodawah commented Oct 19, 2023

I think the number is 722a not 772a.

woops fixed

Do we want to list unrelated cards that just happen to come in the same box? It feels like a different use case than display commanders.

alternatively we could have a special section for them, they just don't really feel "bonus" but something I'd throw away right away 😂

@kodawah
Copy link
Contributor Author

kodawah commented Oct 20, 2023

so this is the wrong place to add the card, the bonus card should go in the product model instead of the deck model and the bonus section should contain the thick commanders instead

@kodawah kodawah closed this Oct 20, 2023
@kodawah kodawah deleted the pctb_bonus branch October 20, 2023 16:29
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.

None yet

3 participants