Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@uekerman
Copy link
Member

In a lecture, a student directly clicked on the link, downloaded the box from the website and did then not know how to go on.
By removing the link, we could avoid this misunderstanding.

Not directly related: Another student reported that they ran into a firewall problem on windows similar to this one maybe. I do not have the error message.
@MakisH Have you seen this one already? Should we add somewhere a statement that maybe users have to tweak their firewall?

In a lecture, a student directly clicked on the link, downloaded the box from the website and did then not know how to go on.
By removing the link, we could avoid this misunderstanding.
@uekerman uekerman requested a review from MakisH January 13, 2022 15:43
Copy link
Member

@MakisH MakisH left a comment

Choose a reason for hiding this comment

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

You are right, this is currently indeed confusing, and this is a much better formulation! 👍 Thanks for the direct edit!

@MakisH
Copy link
Member

MakisH commented Jan 13, 2022

Not directly related: Another student reported that they ran into a firewall problem on windows similar to this one maybe. I do not have the error message. @MakisH Have you seen this one already? Should we add somewhere a statement that maybe users have to tweak their firewall?

I have no idea about this. My gut feeling tells me that the user may have already seen similar errors before (e.g., in Docker) and found this system-specific issue rather easily. But if this is, e.g., in a university network, we should definitely mention it. We could start a (minimized) "troubleshooting" section at the bottom.

@MakisH MakisH merged commit 1a995c2 into master Feb 7, 2022
@MakisH MakisH deleted the uekerman-patch-1 branch February 7, 2022 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

2 participants