Skip to content

Conversation

@rainerhitz
Copy link

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@rainerhitz rainerhitz marked this pull request as draft January 14, 2026 07:48
@rainerhitz rainerhitz marked this pull request as ready for review January 14, 2026 07:52
@wborn wborn requested a review from Copilot January 14, 2026 09:09
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@wborn wborn left a comment

Choose a reason for hiding this comment

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

Perhaps these documentation pages can also be updated/removed now that OR has better Chirpstack and TTS support?

@rainerhitz
Copy link
Author

rainerhitz commented Jan 15, 2026

Perhaps these documentation pages can also be updated/removed now that OR has better Chirpstack and TTS support?

I wouldn't remove these pages because of the following:

LoRa vs. LoRaWAN: The lora documentation focuses on the physical LoRa layer rather than LoRaWAN network servers. It also documents an important student project contribution that we shouldn't lose. I propose updating this page to clarify the distinction between LoRa and LoRaWAN, while adding clear references to the new ChirpStack and TTS documentation.

Manual vs. Automated Config: Regarding the ChirpStack tutorial, it provides a deep dive into configuring MQTT agent links manually. Since our general documentation on manual MQTT agent link configuration is currently limited, this tutorial remains a valuable resource. I can update it to state that it focuses on the manual approach and link it to the newer, more automated ChirpStack/TTS pages.

@wborn
Copy link
Member

wborn commented Jan 15, 2026

while adding clear references to the new ChirpStack and TTS documentation

I can update it to state that it focuses on the manual approach and link it to the newer, more automated ChirpStack/TTS pages.

Would be helpful if you can add these updates so users can more easily find the applicable documentation.

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.

3 participants