Skip to content

support tplink routers#13922

Merged
murrant merged 5 commits intolibrenms:masterfrom
si458:patch-2
Apr 15, 2022
Merged

support tplink routers#13922
murrant merged 5 commits intolibrenms:masterfrom
si458:patch-2

Conversation

@si458
Copy link
Copy Markdown
Contributor

@si458 si458 commented Apr 14, 2022

Please give a short description what your pull request is for

fingers crossed but it now supports any TP-Link router

Please note

Please read this information carefully. You can run ./lnms dev:check to check your code before submitting.

  • Have you followed our code guidelines?
  • If my Pull Request does some changes/fixes/enhancements in the WebUI, I have inserted a screenshot of it.
  • If my Pull Request makes discovery/polling/yaml changes, I have added/updated test data.

Testers

If you would like to test this pull request then please run: ./scripts/github-apply <pr_id>, i.e ./scripts/github-apply 5926
After you are done testing, you can remove the changes with ./scripts/github-remove. If there are schema changes, you can ask on discord how to revert.

@si458 si458 changed the title support tplink TD-W routers support tplink routers Apr 14, 2022
@murrant murrant added Device 🖥️ New or added device support Needs Tests 🦄 https://docs.librenms.org/Developing/os/Test-Units/ labels Apr 14, 2022
@github-actions
Copy link
Copy Markdown

Please add test data so we can ensure your change is not broken in the future.
Read the docs to find out how: https://docs.librenms.org/Developing/os/Test-Units

@murrant murrant removed the Needs Tests 🦄 https://docs.librenms.org/Developing/os/Test-Units/ label Apr 15, 2022
@murrant murrant merged commit 68b49a0 into librenms:master Apr 15, 2022
@si458 si458 deleted the patch-2 branch April 15, 2022 13:09
@librenms-bot
Copy link
Copy Markdown

This pull request has been mentioned on LibreNMS Community. There might be relevant details there:

https://community.librenms.org/t/22-4-0-changelog/18610/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Device 🖥️ New or added device support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants