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

Add WiFi page #15

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add WiFi page #15

wants to merge 6 commits into from

Conversation

Alextopher
Copy link
Member

Where is a good place for this page to live? Is infrastructure/network good ? Should the page be split between two pages ?

Copy link
Contributor

@sophiacarlone sophiacarlone left a comment

Choose a reason for hiding this comment

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

access point (AP)
spelled especially wrong
spelled periodically wrong

@Alextopher
Copy link
Member Author

@JonathanNordby I think I made my last changes to this page. I think you should record a little bit of device information make, model, year of first sale, if applicable the EOL year. And explain how to update the firmware, with sufficient warning to be very careful or you risk bricking the AP

@Alextopher Alextopher removed their assignment Jan 15, 2023
@JonathanNordby JonathanNordby marked this pull request as ready for review March 1, 2023 21:21

## Hardware

// TODO: Equipment page?
Copy link
Member Author

Choose a reason for hiding this comment

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

I think you should record a little bit of device information make, model, year of first sale, if applicable the EOL year.


## Firmware

// TODO: We have a second AP that failed a fireware update and it struggles to boot. Unclear if it's revivable.
Copy link
Member Author

Choose a reason for hiding this comment

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

Form this TODO into a paragraph

@@ -0,0 +1,34 @@
# WiFi

_updated: Jan 14th 2022_
Copy link
Member Author

Choose a reason for hiding this comment

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

Update


We should also periodically check that we are not transmitting on OITs wifi channels. You can do this by using the `Network > Wireless > AP Detection` feature. In general try to keep as much distance as possible in channel selection.

_updated: Dec 7th 2022_
Copy link
Member Author

Choose a reason for hiding this comment

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

Might be worth double checking if you're already signing into the AP

Copy link
Member

@emberian emberian left a comment

Choose a reason for hiding this comment

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

Others: please suggest diffs instead of leaving comments that something needs to be done. It's git! You can just push a commit to the wifi branch. Simply stating that something needs to happen in a comment without even using a line comment makes it more work than necessary for all of us to make progress on this git repo.

@Alextopher
Copy link
Member Author

I only added comments in response to the PR being marked as ready for review. Just to highlight how there are todos left that I'm not going to work on anymore.

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.

5 participants