Skip to content

fix: save principle example b2 notes - #1500

Merged
Zamiell merged 6 commits into
mainfrom
patch-save-principle-example
Jun 25, 2026
Merged

fix: save principle example b2 notes#1500
Zamiell merged 6 commits into
mainfrom
patch-save-principle-example

Conversation

@vEnhance

@vEnhance vEnhance commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

blue cards are missing some possibilities

see recent discussion on Discord #general

the notes don't include b2 as a possibility, so i think the stacks were meant to go up to b2 (or at least this is the simplest way to fix the example)

see recent discussion on Discord #general
@EhDrone

EhDrone commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

That isn't the solution to the example. This is in Save Principle section, if Blue 2 were played then there is no save clue - this is a play clue, and the other cards are noted for the remaining blue cards.

Either the example needs to be reworked, or the all of the cards need negative 2 on them?

@vEnhance

Copy link
Copy Markdown
Contributor Author

ah crap youre right. i'm not thinking.

@vEnhance

Copy link
Copy Markdown
Contributor Author

i guess we can also just edit the notes instead to accommodate b2 possibility? that's a deeper edit so i need to get back to my laptop later for that

@vEnhance vEnhance changed the title fix: save principle example stacks up to b2 fix: save principle example b2 notes Jun 24, 2026
@vEnhance

Copy link
Copy Markdown
Contributor Author

still need to update the main text. once i have my desktop again.

@Zamiell

Zamiell commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

ok, I will just let this one sit then for now, thanks

@vEnhance

Copy link
Copy Markdown
Contributor Author

added some text, hope it's not too wordy? let me know what you think @Zamiell

(also, i accidentally pushed to the main branch at first, sorry about that. i force-pushed it back to where it was before)

do we prefer b2, b3, b4, b5 to b2345? if so i can change that too.

@Zamiell

Zamiell commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

we prefer b2, b3, b4, b5, as it is less cryptic

@vEnhance

Copy link
Copy Markdown
Contributor Author

ok changed

@vEnhance

Copy link
Copy Markdown
Contributor Author

ah, the middle note "b2, b3, b4, b5" is too long here. what's the right way to handle that?

@Zamiell

Zamiell commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

not really sure TBH, i guess we haven't run into that before, i guess use b2345 for now, even though it won't have symmetry with any of the other notes.

@Zamiell
Zamiell merged commit 7061785 into main Jun 25, 2026
10 checks passed
@Zamiell

Zamiell commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

thanks evan, this was a good fix

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