Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Do we care about these? #129

Open
amyschoen opened this issue Feb 10, 2020 · 2 comments
Open

Do we care about these? #129

amyschoen opened this issue Feb 10, 2020 · 2 comments

Comments

@amyschoen
Copy link
Contributor

amyschoen commented Feb 10, 2020

Deleting the template and starting a discussion thread instead. I've been purposely taking the course a bit wrong and I found these issues. Curious if there's desire to check for these or not. I am seeing similar checks when I "make mistakes" 😉 in other parts of the course.

Screen Shot 2020-02-10 at 1 03 23 PM

  • Doing the version number as v0.8 instead of v0.9 still passes.
  • Creating the Project using a template of Basic Kanban instead of Automated Kanban still passes.
  • Dragging the card to the wrong column still passes.

/cc @crichID @brianamarie @hectorsector @JasonEtco

@amyschoen amyschoen added the bug Something isn't working label Feb 10, 2020
@hectorsector
Copy link
Member

Thanks for the report, @amyschoen! My two cents here are that I'd like to block the learner as little as possible -- I've learned the hard way that forcing a learner to do something the right way blocks their progress more often than it helps. So: I try to only do this when it's necessary for the course to work properly (say, if the name of the Release is necessary for the bot to respond).

That being said, I still think we can tackle this by giving the learner feedback, something like:

You didn't name your release v1.0, like I expected, but we'll move on anyway.

@amyschoen
Copy link
Contributor Author

I like that approach @hectorsector.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants