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

RPI Monitor - Custom Addon Titles cannot be hidden #4995

Closed
hackslikeus opened this issue Nov 26, 2021 · 9 comments
Closed

RPI Monitor - Custom Addon Titles cannot be hidden #4995

hackslikeus opened this issue Nov 26, 2021 · 9 comments
Labels
External bug 🐞 For bugs which are not caused by DietPi.

Comments

@hackslikeus
Copy link

hackslikeus commented Nov 26, 2021

Required Information

  • DietPi version | cat /boot/dietpi/.version
    G_DIETPI_VERSION_CORE=7
    G_DIETPI_VERSION_SUB=8
    G_DIETPI_VERSION_RC=2
    G_GITBRANCH='master'
    G_GITOWNER='MichaIng'
    G_LIVE_PATCH_STATUS[0]='applied'
    G_LIVE_PATCH_STATUS[1]='applied'
    G_LIVE_PATCH_STATUS[2]='applied'

  • Distro version | echo $G_DISTRO_NAME $G_RASPBIAN
    buster 1

  • Kernel version | uname -a
    Linux DietPiQAudio1 5.10.63-v7+ #1459 SMP Wed Oct 6 16:41:10 BST 2021 armv7l GNU/Linux

  • SBC model | echo $G_HW_MODEL_NAME or (EG: RPi3)
    RPi 3 Model A+ (armv7l)

  • Power supply used | (EG: 5V 1A RAVpower)
    5V 2.54 A

  • SDcard used | (EG: SanDisk ultra)
    Microcenter class 10 - 16gb

Additional Information (if applicable)

  • Software title | (EG: Nextcloud)
    RPI Monitor

  • Was the software title installed freshly or updated/migrated?
    Installed November 15 (Bullseye), updated, bug noticed, Uninstalled 11/25, Reinstalled 11/26
    apt-update etc. with no impact on issue.

  • Can this issue be replicated on a fresh installation of DietPi?
    Probably

N/A
  • Bug report ID | echo $G_HW_UUID

Steps to reproduce

  1. ...Install Software #66 - RPIMonitor
  2. ...Using either sample addons provided, or create a custom addon, configure in .conf file, attempt to hide the custom
    addon page titleby using "web.addons.X.showTitle=0" or web.addons.showTitle=false" in configuration settings.
    restart RPIMonitor Service, and using the menu option, select the adjusted addon from the menu. The page
    still includes the Addon in the upper left as a page tite, even though it should be hidden.

Expected behaviour

  • ... once addon configuration is set to either 0 (buster) or "false" as stated in example, page title
    should no longer display custom addon name in upper left corner.

Actual behaviour

  • ...Custom Addon Title appears in upper left corner (always) regardles of web.addons.X.showTitle = 0 or false or "false"
@Joulinar
Copy link
Collaborator

similar to the other issue you reported, might be a challenge of RPi-Monitor themselves.

@MichaIng
Copy link
Owner

Yes, please report this upstream, like you did with another issue already: XavierBerger/RPi-Monitor#388
We simply install the latest (soon 4 years old) v2.13 beta6 Debian package from here: https://github.com/XavierBerger/RPi-Monitor-deb

Sadly RPi-Monitor is pretty unmaintained, so I think there won't happen much until someone with JavaScript and perl experience opens an actual PR to fix it.

@MichaIng MichaIng added External bug 🐞 For bugs which are not caused by DietPi. and removed Investigating 🤔 labels Nov 27, 2021
@hackslikeus
Copy link
Author

seems odd that configuration options in dietpi are organized differently: For example, the DietPi data.conf file only lists the individual includes to configuration files found in the templates folder, which is much cleaner compared to the same file
using the installation instructions found on the RPI Monitor readme file.
The data.conf file from the official repository includes a hodge podge of text instructions, alert diagram, and the configuration options for Friends, Header, and most importantly - Addons, not to mention the includes listing.
I was hoping I could pickup something that was missed during the cleanup, as some of the examples files show settings of "false" rather than 0 for false. But after exhausting all local remedies, including uninstall from Dietpi software uninstall, and re-install using the github instructions, the only different outcome was that the ever present page title changed from the Addon name to "undefined", neither of which is the desired outcome.
Will port this issue to the RPI Monitor site as a bug.

@MichaIng
Copy link
Owner

MichaIng commented Nov 27, 2021

Interesting, I wasn't aware that there is an official repository: https://giteduberger.fr/rpimonitor/
So you say that the outcome of installing the package from that repository is different than what dietpi-software ships? That is strange since the above link points as alternative to the exact same package source where we pull the same "latest" package from, so I'd have expected that it is indeed the exact same package.
EDIT: Ah, it might be the latest package from the master branch, which is v2.12 r0, while we pull v2.13 beta6 from the develop branch.

@hackslikeus
Copy link
Author

to clarify my earlier statement re. repository - earlier today I pulled from: here and followed installation instructions, and indeed the data.conf file includes alot of text before the config begins. Just now, I installed from Dietpi software, and the file is concise
Raw_Dietpi_data_conf.txt
.
X_working_data_conf.txt

@MichaIng
Copy link
Owner

Can you verify that it is v2.12 when following the instruction (installing from official repository)?

@hackslikeus
Copy link
Author

hackslikeus commented Nov 27, 2021 via email

@MichaIng
Copy link
Owner

Okay, then the cleaned up configuration file is part of v2.13. Although I have to say that I generally like config files which contain all available settings and comments to explain them, so one does not need to check documentations elsewhere 😄. But it's fine, good when you like it more the slim way.

@hackslikeus
Copy link
Author

hackslikeus commented Nov 28, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
External bug 🐞 For bugs which are not caused by DietPi.
Projects
None yet
Development

No branches or pull requests

3 participants