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

Allow scrolling the world map unless scrollable: false is set #1109

Merged
merged 1 commit into from Feb 15, 2023

Conversation

byorgey
Copy link
Member

@byorgey byorgey commented Feb 14, 2023

Adds a new boolean key scrollable to the world description in scenario files, with a default value of true.

Closes #1108.

Adds a new boolean key `scrollable` to the world description in
scenario files, with a default value of true.

Closes #1108.
@byorgey
Copy link
Member Author

byorgey commented Feb 15, 2023

I don't know what happened with the TODO checking. I got an email saying that it didn't run because it was empty, or something like that, and it seems not to be listed among the checks above.

Copy link
Member

@xsebek xsebek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the JSON schema as well. 👍

@xsebek
Copy link
Member

xsebek commented Feb 15, 2023

@byorgey could you please paste the message here or forward me the email?

I will check why the TODO was not run. 🤨

@byorgey byorgey added the merge me Trigger the merge process of the Pull request. label Feb 15, 2023
@mergify mergify bot merged commit 0ef8fc7 into main Feb 15, 2023
@mergify mergify bot deleted the feature/scrollable branch February 15, 2023 10:56
@byorgey
Copy link
Member Author

byorgey commented Feb 15, 2023

@xsebek I can forward you the email, but I don't think I have your email address. You can PM it to me on IRC or something.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge me Trigger the merge process of the Pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scrolling map in challenge scenarios
3 participants