search

YunoHost-Apps / aeneria_ynh

æneria package for YunoHost
GNU Affero General Public License v3.0
11 stars 2 forks source link

Can't access to configuration page #16

Closed urschuca closed 3 years ago

urschuca commented 3 years ago

Describe the bug

I can't access to the configuration page

Context

# yunohost app info aeneria
description: Dashboard to analyse your energy consumption data from Linky & weather
domain_path: urschuca.com/aeneria
name: aeneria
version: 1.1.3~ynh1

Steps to reproduce

Each time I click on the configuration links I get 

Oups'

Désolé, une erreur est survenue !

Si cette erreur persiste, merci de nous contacter.

Expected behavior

Access to the configuration page

Logs

# tail /var/www/aeneria/var/log/prod-php-2021-08-04.log
...
[2021-08-04T13:53:44.876312+00:00] php.CRITICAL: Uncaught Error: Return value of App\Twig\AppExtension::getLinkyDescription() must be an instance of Aeneria\EnedisDataConnectApi\Model\Address or null, string returned {"exception":"[object] (TypeError(code: 0): Return value of App\\Twig\\AppExtension::getLinkyDescription() must be an instance of Aeneria\\EnedisDataConnectApi\\Model\\Address or null, string returned at /var/www/aeneria/src/Twig/AppExtension.php:150)"} []

edit : formating

SimonMellerin commented 3 years ago

Thanks for reporting this, I'm gonna check that

kedare commented 3 years ago

I confirm I have the exact same issue.

NoWiS- commented 3 years ago

Hello,

A fix already exist https://gitlab.com/aeneria/aeneria-app/-/commit/3e0de977a99e75c63a2ec873e848e60b39f99528 (I applied it manually , and it works perfectly. The path is /var/www/aeneria/src/Twig/AppExtension.php)

There's already a pull request https://github.com/YunoHost-Apps/aeneria_ynh/pull/15 to bump the version of aeneria to 1.1.5. Can I do something to help its approval ?

SimonMellerin commented 3 years ago

Right, don't know why this PR haven't been merged yet.

I've merged it.

Let's wait for tests to be ok here : https://github.com/YunoHost-Apps/aeneria_ynh/pull/17 and then it should be ok to merge it to master

SimonMellerin commented 3 years ago

Should be ok now with last update.

@kedare or @NoWiS- Feel free to close this issue if it's ok for you

NoWiS- commented 3 years ago

I updated my instance, and it works, thanks !

(I haven't however the right to close the issue)