-
Notifications
You must be signed in to change notification settings - Fork 17
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
Labs 2,3,4,5 submission #12
base: main
Are you sure you want to change the base?
Conversation
Good work. You can also add an autostart using the 'after' construct in xstate. Also, 'guard' is the xstate 5 terminology, while they were called 'cond' in xstate 4 and earlier. I'm not sure how backwards compatible the new version is, so perhaps this is the source of your cond/guard problems. |
79e44e6
to
389c1aa
Compare
f89ca27
to
96cbfe4
Compare
Comments lab 4: It's a bit surprising that your submission does not implement the flow chart from lab 2, but your submission still basically shows that you can do what was asked of you, so I'm going to pass you. Comments lab 5: The system crashes on out-of-grammar meeting titles. This must be fixed, since you can not expect the user to know in advance exactly what meeting titles would be accepted, and making sure that out-of-grammar situations are handled correctly is one of the main tasks of lab 5. |
thanks for your @RasmusBlanck reply! I have modified the grammar section and test them. I think 'not in grammar' works right now. Could you help test them again? |
No description provided.