-
Notifications
You must be signed in to change notification settings - Fork 397
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
Implement Recurring and Placed Credits #4262
Merged
Merged
Changes from 1 commit
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
f83bc2b
First approach to implementing a credits prompt
lostgeek 14baef0
Change to make-eid to reuse fields from old eid
lostgeek 935a996
pay-credits working with recurring credits
lostgeek ab2e911
Structure of :interactions :pay-credits changed
lostgeek 7439330
pick-credit-providing-cards now calling provider-func only once
lostgeek d96b699
Comparing :cid instead of whole cards
lostgeek 5846ee4
Fixed order of args on pick-virus-counters-to-spend
lostgeek cd43714
pay-credits now works on paid-abilities
lostgeek 7f95a68
After using pay-credits, rest will now be paid from credit pool
lostgeek e761a42
I accidentally a verb
lostgeek 5be62e7
Added pay-credits prompts to a bunch of cards
lostgeek 71711f4
pay-credits on playing Events (and Operations)
lostgeek a17bd01
Reverted eid change in move
lostgeek c7fa595
Added pay-credits for removing tags
lostgeek 6d95403
Trim whitespace on remove tag message
lostgeek 8db6dd5
Added effect-completed for remove-tag
lostgeek 32c1ed3
Refactor and speed up pick-credit-providing-cards
NoahTheDuke 7b6f089
Merge pull request #1 from NoahTheDuke/lostgeek-pay-credits-prompt
lostgeek 9650c9d
Implemented Noahs comments
lostgeek 45c4381
Added pay-credits prompt to rezzing cards
lostgeek 6fef990
Swapped The Root with Off the Grid to avoid pay-credits prompt
lostgeek 027e3ce
Removed test-refresh...
lostgeek c48cf98
Added pay-credits prompt to installing corp cards
lostgeek 4bce666
Added pay-credits prompt to advancing cards
lostgeek 2084354
Added pay-credits to Trickster Taka and Cold Read
lostgeek 2abc341
Implemented pay-credits prompt for Flame-out
lostgeek 3e1acd0
Removed unnecessary do
lostgeek 4778e80
Implement pay-credits prompts for trashing accessed cards
NoahTheDuke 534ad1c
Implement pay-credit prompts for traces
NoahTheDuke ef4a8da
Added pay-credits prompt to Patchwork. Reworked :custom functions
lostgeek 4256ba1
Merge branch 'pay-credits-prompt' into lostgeek-pay-credits-prompt
lostgeek 5592046
Merge pull request #2 from NoahTheDuke/lostgeek-pay-credits-prompt
lostgeek 2a48ca4
Fixed bugs introduced in the merge...
lostgeek 02a8dab
play-ability now works correctly with can-pay?
lostgeek 3889041
Tests for pay-credits prompts on installing cards as Runner and Steal…
lostgeek 231a201
Tests for pay-credits prompts on paid abilities on icebreakers and cr…
lostgeek 2385781
Tests for pay-credits prompt on playing events and removing tags
lostgeek d9e0005
Tests for pay-credits prompt on rezzing cards. Fixed
lostgeek 51637cd
Fixed previous tests that used `changes-credits`
lostgeek 5777e51
Tests for pay-credits prompts on installing and advancing Corp cards
lostgeek 608e874
Fixed Thunder Art Gallery + Sahasrara + pay-credits interaction
lostgeek 5ed94de
Clean up custom pay-credit effects, cold read
NoahTheDuke d06167a
Merge pull request #3 from NoahTheDuke/lostgeek-pay-credits-prompt
lostgeek 23a91b5
Net Mercur now shows pay-credits prompt during runs and traces
lostgeek File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since removing tags is now awaitable, I should test a case of Sahasrara + Thunder Art Gallery to check if all the prompts are handled correctly.