update release & script, and move admin creation before daemon

[mayel]

No description provided.

[mayel]

hold on, moving the admin creation means it doesn't have .env loaded, working on that

[mayel]

@lapineige can we test this?

[lapineige]

!testme

[yunohost-bot]

✌️

[mayel]

hmm, so maybe the environment of the install script is not shared with the app when run by ynh_exec_warn_less

[mayel]

the reason I moved it to before the daemon (and using eval which starts the app, instead of rpc which tries to connect to an already running app) is that otherwise without some way to know the app has finished starting and is in a ready state the user creation may fail to early

[lapineige]

otherwise without some way to know the app has finished starting and is in a ready state the user creation may fail to early

The start command config should be there to wait any "I finished starting" line. Waiting for that log : https://github.com/bonfire-networks/bonfire-yunohost/blob/use-release/scripts/install#L126

[mayel]

aha!

[lapineige]

!testme

[yunohost-bot]

Alrighty!

[yunohost-bot]

📚 🪱

[lapineige]

Same errors apparently.
These messages are quite confusing for me :(

[lapineige]

Could you update to latest release https://github.com/bonfire-networks/bonfire-app/releases/tag/v0.9.10-beta.75 ?
I can't edit the branch myself.

[mayel]

@lapineige done :)

[lapineige]

!testme

[yunohost-bot]

May the CI gods be with you!

[yunohost-bot]

Living in the future, are we?

[lapineige]

Already a new release in the meantime 😅
Is that necessary for the adsf just issue ?