Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

💻 Shorten our error messages #5012

Closed
MarleenGilsing opened this issue Jan 18, 2024 · 1 comment
Closed

💻 Shorten our error messages #5012

MarleenGilsing opened this issue Jan 18, 2024 · 1 comment
Labels
error messages Issues related to Error messages when code is wrong frontend Issue with the web interface

Comments

@MarleenGilsing
Copy link
Collaborator

Our error messages might be longer than necessary. For instance:
image

This error message doesn't need 'at position 13'. I don't think students know what we mean by position 13 anyway and the mistake is highlighted nowadays, so the student will instantly see what they did wrong in line 1.

I would even argue that 'The code you entered is not valid Hedy code.' is unnecessary information.

I think the shorter the message, the bigger change of a kid actually reading the message, so maybe we can take a look at the messages and tweak all the extra info.

@MarleenGilsing MarleenGilsing added error messages Issues related to Error messages when code is wrong frontend Issue with the web interface labels Jan 18, 2024
@Felienne
Copy link
Member

Great remark! I have made similar remarks in #4729 Let's discuss this at the hackathon?

@hedyorg hedyorg locked and limited conversation to collaborators Feb 7, 2024
@Felienne Felienne converted this issue into discussion #5079 Feb 7, 2024

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
error messages Issues related to Error messages when code is wrong frontend Issue with the web interface
Projects
Status: Soon
Development

No branches or pull requests

2 participants