Feat/daemon selection

[janepie]

Resolves #276

I had to prolong the waiting time for the registering as (at least on my setup) it took 15-20 sec (also when done per occ command directly, I checked)

Did not comment the Vue parts as you don't seem to use comments there. If I should add comments just let me know!

[bigcat88]

Thanks for the pull request, it's good, but there are a couple of small inaccuracies that will be good to be fixed:

1. Daemons with the type "manual-install" should not be displayed
2. If there is only one Daemon (after filter by "manual-install"), the selection window should not appear
3. When the selection window opens, you see the text "Selection Modal" at the top center - it should be removed (see on attached screen)

[janepie]

Aah yes, thank you, I will look into that tomorrow!

[andrey18106]

It seems there are some syntax errors in new components (missing prop, error prop type check) in console while opening daemon selection - I don't see the list of daemons to choose from (I have one manual-install and one docker-install daemon).

[janepie]

Then you shouldn't see the list as it's only one valid daemon. Can you check with two docker-install?

Edit: ah damn, I see what you mean, will check

[janepie]

Should be fixed, but something in my NC instance seems broken right now and I cannot validate. I will try to fix it and check, feel free to check yourself or wait :)

[janepie]

Works!

[bigcat88]

Works!

Great, it almost works, there is a few last little details left.

When the application is disabled, the window for selecting a daemon to enable should not appear, since this window is for choosing where to install the application (on which daemon to deploy)
When enabling the application, we do not need to display it.

Steps to reproduce:

1. Click "Deploy and Enable"
2. Once the deploying is complete, click "Disable"
3. After clicking "Enable", the ModalWindow should not appear and we just need to enable it

---

https://github.com/nextcloud/app_api/pull/394/files#diff-07f98afc574e5706d7ad885a1c359bf699dd1045b0cfcf54e325110c9f45a0aaR189

this string executes and make request to the server for each ExApp on these tabs(you can see this in Network tab in browser):

Is there a way to fix this so we can consolidate PRs and consider all tasks complete and done?

[janepie]

Totally, both points done :)

[bigcat88]

Thank you for putting effort on this, great job.

[bigcat88]

now it’s up to you, Andrey, when we will combine this and whether we will make adjustments on our part.