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

Patch 1 #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Patch 1 #5

wants to merge 2 commits into from

Conversation

sistein
Copy link

@sistein sistein commented Mar 2, 2019

I had a noncritical problem while updating a 4020. It was not found by the script. I hav several network adaptors activated which might have created this problem. After disabling everything was fine.

I added the section to make it easier for a non techi person to update.

Translation to German is missing since I'm not familiar with this babel stuff.

@blocktrron blocktrron self-assigned this Mar 2, 2019
@blocktrron
Copy link
Member

I've adjusted the wording and added german translations. See https://github.com/freifunk-darmstadt/fritz-tools/tree/pr-win10-network-adaptors

However, i'm a bit unhappy with how this section is placed in the tutorial. We take the "Click here - click there" approach, which the current placement within the documentation violates. Also the concrete steps on how to activate/deactivate the adapter is not covered.

With this in mind, where would you place the instructions?

@sistein
Copy link
Author

sistein commented Mar 3, 2019

Perhaps we could combine it with the wifi section. Actually if the adaptors are diasbled wifi will be disabled too. I can also add some more pictures showing how to enable/disable the adaptors.

@blocktrron
Copy link
Member

I would clearly mark the "Disable other adapters" part as optional as i already see inexperienced users turn off all their adapters. Maybe add your section on the bottom as a step of troubleshooting?

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

Successfully merging this pull request may close these issues.

None yet

2 participants