Skip to content

Issue fixes #79

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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

Issue fixes #79

wants to merge 20 commits into from

Conversation

kgalb01
Copy link
Contributor

@kgalb01 kgalb01 commented Apr 2, 2025

Fixed most of the issues of the github repository with 3 exceptions, which are lower on the priority list (and take significantly longer to fix).

The three remaining issues are:

  • Add tutorial how to add Home Assistant to openSenseMap
  • Neugestaltung der Übersichtsseite
  • Vergleich von unseren Boards

I will take care of them at a later point in time.

closes #29
closes #76
closes #14
closes #77
closes #69
closes #65

various dead links have been fixed and other smaller erros

Copy link

netlify bot commented Apr 2, 2025

Deploy Preview for docs-sensebox-de ready!

Name Link
🔨 Latest commit 73ed291
🔍 Latest deploy log https://app.netlify.com/projects/docs-sensebox-de/deploys/682745c81118ad0009aef2e4
😎 Deploy Preview https://deploy-preview-79--docs-sensebox-de.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@Thiemann96 Thiemann96 left a comment

Choose a reason for hiding this comment

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

Good first draft, please fix merge conflicts and comments above ! 👍

@@ -79,7 +79,8 @@ const config = {
({
colorMode: {
defaultMode: 'light',
Copy link
Contributor

Choose a reason for hiding this comment

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

For some reason, the default mode for me is the dark mode

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the default mode is always the preferred mode of the user. I tried it with different devices, and the default mode for devices where light mode is activated is also light mode. I can deactivate that, though, so that light mode is always the default.

</PopoverTrigger>
<PopoverContent>
<div>
<h2>Sensors and accessories on the I2C/Wire ports</h2>
Copy link
Contributor

Choose a reason for hiding this comment

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

The popover is now buggy for me
grafik

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@@ -17,17 +17,76 @@ import TutorialPorts from '@site/src/components/TutorialPorts/TutorialPorts'
- 12x8 Matrix
- 96 LEDs

---

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe have a short introductory text to the led matrix and what can be done with it, e.g. other sensors like HDC1080

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

Copy link
Contributor

Choose a reason for hiding this comment

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

Only show cables which are in the selected board. e.g. with edu-s2 selected the JST to JST cables should not show up.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed


Je nach Anwendung und Sensorposition kannst du die passende Länge wählen.

<figure style={{ maxWidth: '400px', marginBottom: '1.5rem' }}>
Copy link
Contributor

Choose a reason for hiding this comment

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

Lets try to use the Component. This way images have a subtitle, are evenly sized and centered

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed


Nahezu alle **neueren Sensoren** der senseBox verfügen sowohl über einen **QWIIC**- als auch über einen **JST PHR-4 Anschluss**, um sowohl mit der **MCU S1** als auch mit der **MCU S2** kompatibel zu bleiben. Langfristig wird QWIIC jedoch als Hauptstandard verwendet.

> ⚠️ **Wichtiger Hinweis zur Versorgungsspannung**:
Copy link
Contributor

Choose a reason for hiding this comment

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

Use the same alert box you used on the mcu-s2 overview page

grafik grafik

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants