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

[Vision] fix intangible cost and double side display #464

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mmougy
Copy link
Contributor

@mmougy mmougy commented Feb 8, 2024

  • fix Intangible cost
    and display the double side Intangible/Dense like Solid/Phased

  • add illustrator for the B side

@jamiehumphries
Copy link
Contributor

Is there a reason that this PR hasn't been merged yet? It's quite a bit older than the other open ones.

If there is anything that needs further attention on it, I'd be happy to pick it up.

@mmougy mmougy marked this pull request as draft November 25, 2024 20:39
@mmougy mmougy force-pushed the fix-vision-intangible-cost branch from 5a83f9a to 148df95 Compare December 29, 2024 20:43
@mmougy mmougy force-pushed the fix-vision-intangible-cost branch from 148df95 to 4ecc787 Compare December 29, 2024 21:08
@mmougy mmougy marked this pull request as ready for review December 29, 2024 21:11
@mmougy
Copy link
Contributor Author

mmougy commented Jan 24, 2025

here is the result from my local dev environment

image

If this fix is applied, the card https://marvelcdb.com/card/26002 should be deleted from the DB

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.

2 participants