owncloud / ocis

:atom_symbol: ownCloud Infinite Scale Stack
https://doc.owncloud.com/ocis/next/
Apache License 2.0
1.4k stars 182 forks source link

Custom web + WOPI example broken? #6531

Closed pascalwengerter closed 1 year ago

pascalwengerter commented 1 year ago

Describe the bug

I've tried following https://owncloud.dev/ocis/deployment/ocis_wopi/ to change my changes from https://github.com/owncloud/web/pull/8447, but when linking my own /web/dist seems to mess with the registration of appProvider app actions?

Expected behavior

Guide should work flawlessly ™️

Actual behavior

Additional context

Pulled fresh images and deleted volumes before starting docker. I could try and debug this myself, but maybe I'm missing something obvious/undocumented (like linking/modifying a config file)?

pascalwengerter commented 1 year ago

Following a discussion with @kulmann that may be a Mac M1 + docker issue (on Docker Desktop v4.1{5-9})

kulmann commented 1 year ago

Following a discussion with @kulmann that may be a Mac M1 + docker issue (on Docker Desktop v4.1{5-9})

Yes, I had the same issues. Would be great if someone with a linux system could check as well before any other debugging takes place.

mmattel commented 1 year ago

@micbar just an idea to think about, maybe not related to this issue but could be:

Atm we have deployment examples with a particualr state in various tags. What we dont have is a kind of release notes file in the main directory where we just summarize changes made from one to another version. What about creating one which would ease finding differences? Note that these examples are not only available for devs but also linked by the admin docs.

micbar commented 1 year ago

@pascalwengerter Which version are you using?

Do you have my changes included https://github.com/owncloud/ocis/commit/a86aba1c4144d54d80fd956ddcb9139518cbfc04 ?

pascalwengerter commented 1 year ago

@pascalwengerter Which version are you using?

Do you have my changes included a86aba1 ?

Yes, on my local oCIS the most recent commit was on June 14th (3ce2413c8026aa2469dae3399077c693a7f67ab8)

micbar commented 1 year ago

Can you check what the /app/list request returns?

pascalwengerter commented 1 year ago

Can you check what the /app/list request returns?

{"mime-types":[]}

pascalwengerter commented 1 year ago

I'll close here since I retried this morning (updated oCIS and docker-for-mac before), and if you wait long enough the /app/list endpoint does return the extensions...it just takes quite a while, which I didn't expect