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

philmel: cleanup #893

Merged
merged 1 commit into from
Dec 16, 2024
Merged

philmel: cleanup #893

merged 1 commit into from
Dec 16, 2024

Conversation

FFHener
Copy link
Contributor

@FFHener FFHener commented Jul 16, 2024

I had a meeting with @patrella and we definied together how the location will look like after the maintenance work.

Please review, but do not merge as it doenst reflect the current state. I will merge it after the on site work is done

@FFHener FFHener force-pushed the philmel-cleanup branch 2 times, most recently from b72b78e to 18da6a9 Compare July 26, 2024 11:06
@FFHener FFHener marked this pull request as ready for review July 26, 2024 11:40
@FFHener FFHener requested review from noxilixon and booo July 29, 2024 17:55
Copy link
Contributor

@noxilixon noxilixon left a comment

Choose a reason for hiding this comment

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

👍

locations/philmel.yml Outdated Show resolved Hide resolved
locations/philmel.yml Outdated Show resolved Hide resolved
locations/philmel.yml Outdated Show resolved Hide resolved
locations/philmel.yml Outdated Show resolved Hide resolved
@FFHener FFHener marked this pull request as draft July 30, 2024 12:04
@FFHener
Copy link
Contributor Author

FFHener commented Aug 27, 2024

@Petrella are you up for making this PR ready so we can merge it?

@FFHener FFHener force-pushed the philmel-cleanup branch 2 times, most recently from 5ed1d01 to a62b57c Compare October 21, 2024 17:05
@Noki Noki mentioned this pull request Nov 1, 2024
@FFHener FFHener force-pushed the philmel-cleanup branch 2 times, most recently from c5d63fd to b86e7a2 Compare November 13, 2024 19:19
@FFHener FFHener marked this pull request as ready for review November 13, 2024 19:19
@FFHener FFHener requested a review from noxilixon November 13, 2024 20:05
locations/philmel.yml Outdated Show resolved Hide resolved
locations/philmel.yml Outdated Show resolved Hide resolved
locations/philmel.yml Outdated Show resolved Hide resolved
@FFHener FFHener force-pushed the philmel-cleanup branch 2 times, most recently from 77b2e41 to a82fe78 Compare November 21, 2024 08:46
@FFHener FFHener requested a review from noxilixon November 21, 2024 08:46
@Noki
Copy link
Member

Noki commented Dec 6, 2024

the idea with the custom wireless profile for AP meshing did not work, we should revert it and merge and simply help the guy to upgrade the router to using 802.11s.

Switched IPs
Switched mgmt-vlan to 438
Added nearfield devices
Added frequency management
Replaced some Devices
@FFHener
Copy link
Contributor Author

FFHener commented Dec 16, 2024

Removed the commit. Merging as it was approved before

@FFHener FFHener merged commit d2c73b0 into main Dec 16, 2024
11 checks passed
@FFHener FFHener deleted the philmel-cleanup branch December 16, 2024 13:48
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.

4 participants