-
Notifications
You must be signed in to change notification settings - Fork 37
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
Custom Deck Support #13
Comments
How about adding another character (e.g. https://github.com/crhallberg/json-against-humanity/blob/master/compile.py#L35-L40 I'm happy to submit a PR if you agree with the approach. |
An alternative would be to use a special comment at the beginning or end of the line for this, |
I like the special comment idea... maybe just check if a line of the file is a JSON object or not so it can be:
|
Can do, but then the individual files are not really markdown anymore. If we stick to comments
To be fair, comments aren't really markdown either but many processors (including GitHub flavored markdown) allow them. @crhallberg please advise :) |
The haiku issue is resolved with v2 (see cah-all-compact.json). Renaming this issue to move the conversation to custom deck support. |
To continue the conversation, would it be a good choice to support CSV? Blank third column counts underscores? Maybe with a metadata JSON?
|
As a heads up, we are consuming JSON Against Humanity data as a part of Massive Decks, and have a repository of custom decks people make in Many Decks. Our format has some features that wouldn't be supported by an ingestion format like that, and while I can understand if you just don't want to support them, I figure I'll give them here for context:
You can see example of this in this issue. As I say, I get if these are just out of scope for the format, but if they were supported it would be cool to use the same ingestion format for Many Decks' import functionality. (As a side note: it might be nice to try and do some integration if you were open to it, e.g: the ability to submit decks built in Many Decks to JaH directly, or at least the ability to export to this format to make it easier). |
Hey, @Lattyware! Lots to think about! While I imagined the custom deck support being a local compilation, your use case is definitely one to strongly consider. I looked through your links and recommendations and I had a couple of ideas:
{
"text": "_?! I **HATE** _!!",
"pick": 1,
"slots": [
{ "index": 0, "transform": "UpperCase" },
{ "index": 0 },
]
}
|
Adding a CLI argument to include custom decks is a change I really want to include.
node update-from-sheets.js --add custom.csv
There's a discussion below about supporting custom black cards without counting underscores.
The text was updated successfully, but these errors were encountered: