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

Adds Width and Height-specific on-board Card Scaling #59

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Zetal911
Copy link

Also adds a more verbose error message when a player attempts an action that requires a GM, if a GM is not connected.

I wound up needing to do some edits to the gulpfile and update some packages in order to get the build task to work. This is my first time working with Gulp, though, so that might be a result of me doing something wrong instead.

@Norc
Copy link
Collaborator

Norc commented May 22, 2021

Hey, sorry, Spaceman's been busy with other projects and I just noticed there were PRs here.

I'm going to hold off on merging this, particularly with the changes to the Gulp being involved. Also, while your changes to improve GM handling seem helfpul, I'm not sure that it calls for full author credit. With that said, I'm entirely unclear on open source ettiquette.

Thanks for the PR!

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