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

Tutorial improvements #1067

Merged
merged 3 commits into from Jan 31, 2023
Merged

Tutorial improvements #1067

merged 3 commits into from Jan 31, 2023

Conversation

byorgey
Copy link
Member

@byorgey byorgey commented Jan 31, 2023

A few miscellaneous tutorial improvements.

  • In the bind tutorial, give the base a compass (in inventory) and logger (installed).

    There's no reason not to allow a compass in this challenge, if people had become comfortable typing things like turn north.

    Also, the base absolutely ought to have a logger equipped. As it is, if the user tries to build a robot that does something the base does not have the devices for (such as turn east, before), it simply fails silently.

  • Give the base some compass devices in require tutorial. Again, no reason we need to wean players off the compass quite yet.

The "first steps" tutorial is the first place where the player won't have a compass. I don't know if it's worth calling that out explicitly in the goal text, or if we just let the player figure it out.

There's no reason not to allow a `compass` in this challenge, if
people had become comfortable typing things like `turn north`.  Later
we will have to break it to them that they will have to do without
this for a while...

Also, the base absolutely ought to have a `logger` equipped.  As it
is, if the user tries to `build` a robot that does something the base
does not have the devices for, it simply fails silently.
Again, no reason we need to wean players off the `compass` quite yet.
@byorgey byorgey requested review from kostmo and xsebek January 31, 2023 20:56
Copy link
Member

@kostmo kostmo left a comment

Choose a reason for hiding this comment

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

lgtm

@@ -77,10 +77,12 @@ robots:
attr: robot
devices:
- 3D printer
- logger
Copy link
Member

Choose a reason for hiding this comment

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

We should test that all tutorial scenarios have a logger otherwise players will miss out on key help features. 😱

@byorgey byorgey added the merge me Trigger the merge process of the Pull request. label Jan 31, 2023
@mergify mergify bot merged commit e02d54a into main Jan 31, 2023
@mergify mergify bot deleted the tutorial-improvements branch January 31, 2023 22:00
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.

None yet

3 participants