Skip to content

Add URL Display Theme for Raspberry Pi#3205

Open
MartinRinas wants to merge 49 commits intoopenWB:masterfrom
MartinRinas:copilot/implement-display-theme-modbus
Open

Add URL Display Theme for Raspberry Pi#3205
MartinRinas wants to merge 49 commits intoopenWB:masterfrom
MartinRinas:copilot/implement-display-theme-modbus

Conversation

@MartinRinas
Copy link
Contributor

@MartinRinas MartinRinas commented Mar 15, 2026

Introduce a new URL display theme for Raspberry Pi, enabling users to connect to a specified URL.

MartinRinas and others added 30 commits June 28, 2023 18:25
@MartinRinas
Update publish_docs_to_wiki.yml
19a9835
@MartinRinas
Merge branch 'openWB:master' into master
07abf0b
@MartinRinas
Merge branch 'openWB:master' into master
1ae671a
@MartinRinas
Merge branch 'openWB:master' into master
2646d1a
@MartinRinas
Merge branch 'openWB:master' into master
f8944c5
@MartinRinas
Merge branch 'openWB:master' into master
e83795f
@MartinRinas
Merge branch 'openWB:master' into master
c4a0d32
@MartinRinas
Merge branch 'openWB:master' into master
14694d4
@MartinRinas
Merge branch 'openWB:master' into master
8afa5c2
@MartinRinas
Merge branch 'openWB:master' into master
6be1634
@MartinRinas
Merge branch 'openWB:master' into master
b6074a1
@LKuemmel @MartinRinas
fems: support multiple and single segment regex rqeuests (openWB#1397)
6f581a8 
* fems: support multiple and single segment regex rqeuests

* fix
@LKuemmel @MartinRinas
one connect to server for all components
719a191
@LKuemmel @MartinRinas
0.1s sleep between readings
0387eed
@dj3mu @MartinRinas
fix self.data --> self._data in fems/bat.py
e033d36
@LKuemmel @MartinRinas
Duo+RFID: fix matching if other cp is plugged
5773c81
@LKuemmel @MartinRinas
Revert "fix rct crashes"
fd411b4
@LKuemmel @MartinRinas
RCT: 0.5s sleep between server connections
9577b56
@LKuemmel @MartinRinas
build UI
e360cb8
@MartinRinas
Calculate_soc_tronity (openWB#1398)
59e83fa 
* Update publish_docs_to_wiki.yml

* enable calculate soc for Tronity

* fix
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core
c32d6d8
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core
909fb3f
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core
3aa4283
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core
9d9c7b3
@MartinRinas
Merge branch 'openWB:master' into master
17dcaea
@MartinRinas
Merge branch 'openWB:master' into master
65e9b0e
@MartinRinas
Merge branch 'openWB:master' into master
9d78bd5
@MartinRinas
Merge branch 'openWB:master' into master
d936545
@MartinRinas
Merge branch 'openWB:master' into master
13cc4de
@MartinRinas
Merge branch 'openWB:master' into master
068bf56
MartinRinas and others added 15 commits December 18, 2024 14:20
@MartinRinas
Merge branch 'openWB:master' into master
27cde5d
@MartinRinas
Merge branch 'master' of https://github.com/MartinRinas/core
68261e6
@MartinRinas
Merge branch 'openWB:master' into master
baccc10
@MartinRinas
Merge branch 'openWB:master' into master
7bf97bb
@MartinRinas
Merge branch 'openWB:master' into master
0596bab
@MartinRinas
Merge branch 'openWB:master' into master
07389a4
@MartinRinas
Merge branch 'openWB:master' into master
c1c6610
@MartinRinas
fix publish soc_timestamp
bd49e28
Initial plan
b41f1d9
@MartinRinas
Add URL Display Theme for Raspberry Pi
34cdd45 
Co-authored-by: MartinRinas <29856849+MartinRinas@users.noreply.github.com>
@MartinRinas
Fix insecure randomness: replace Math.random() with crypto.randomUUID…
661d0f2 
…() for MQTT client ID

Co-authored-by: MartinRinas <29856849+MartinRinas@users.noreply.github.com>
@MartinRinas
Revert "Fix insecure randomness: replace Math.random() with crypto.ra…
61b54a0 
…ndomUUID() for MQTT client ID"

This reverts commit 661d0f2.
@MartinRinas
symlink
52c8ac0
@MartinRinas
symlink
77da3a6
@MartinRinas
Merge branch 'master' of https://github.com/openWB/core into copilot/…
1bb067e 
…implement-display-theme-modbus
benderl
benderl requested changes Mar 16, 2026
View reviewed changes
Copy link
Contributor

@benderl benderl left a comment

Choose a reason for hiding this comment

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

Danke für das Display Theme.

@benderl benderl mentioned this pull request Mar 16, 2026
Open
@benderl benderl linked an issue Mar 16, 2026 that may be closed by this pull request
Feature Request: Show alternative website in integrated display #1975
Open
@benderl benderl self-assigned this Mar 16, 2026
@benderl benderl added the enhancement New feature or request label Mar 16, 2026
@MartinRinas
Copy link
Contributor Author

MartinRinas commented Mar 16, 2026

@benderl danke fürs Review, hab' jetzt auch eine mini-build pipeline via package.json und ein wenig Validierung gegen die URL.
image

@benderl
Copy link
Contributor

benderl commented Mar 16, 2026

Sieht auf jeden Fall professioneller aus, als vorher. 👍
Gibt es einen Grund, warum Du die alte 6.x Version von Vite verwendest? Bei einem neuen Projekt wäre ich mindestens auf 7.x, wenn nicht sogar die brandneue 8.x gegangen. Das erspart in Zukunft ein bzw. zwei Migrationen.

@MartinRinas
Copy link
Contributor Author

MartinRinas commented Mar 16, 2026

Sieht auf jeden Fall professioneller aus, als vorher. 👍 Gibt es einen Grund, warum Du die alte 6.x Version von Vite verwendest? Bei einem neuen Projekt wäre ich mindestens auf 7.x, wenn nicht sogar die brandneue 8.x gegangen. Das erspart in Zukunft ein bzw. zwei Migrationen.

hm im Grunde nicht, ich werde meinen Mitarbeiter Copilot da noch mal drauf ansprechen. Kann mir keinen Grund vorstellen da nicht gleich auf die aktuellse Version zu gehen.

@benderl
Copy link
Contributor

benderl commented Mar 17, 2026

Im Ordner .github/workflows kannst Du analog zu den anderen Display Themes eine Datei check_display_theme_url_display.yml anlegen.

Die Build-Dateien unter "web" gehören nicht in den PR. Das Theme wird automatisch nach dem Merge neu gebaut. Das wird in der Datei .github/workflows/publish_to_master.yml erledigt. Voraussetzung ist, dass es eine Aktion "build" in der package.json Datei gibt, was hier der Fall ist.

@MartinRinas MartinRinas mentioned this pull request Mar 19, 2026
Open
@benderl
Copy link
Contributor

benderl commented Mar 20, 2026

Wenn Du mit dem Theme fertig bist, bitte melden.

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

Reviewers

@benderl benderl benderl requested changes

Assignees

@benderl benderl

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feature Request: Show alternative website in integrated display

5 participants

@MartinRinas @benderl @LKuemmel @dj3mu