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

Files external app is abusing Namespace of the External app #11945

Closed
nickvergessen opened this issue Oct 19, 2018 · 2 comments
Closed

Files external app is abusing Namespace of the External app #11945

nickvergessen opened this issue Oct 19, 2018 · 2 comments

Comments

@nickvergessen
Copy link
Member

The JS code of the files_external app must not be in OCA.External.
That namespace belongs to the external app:

OCA.External.App = {

https://github.com/nextcloud/external/blob/6d96ef757933a2fe3527da50e6e3831f93104b50/js/admin.js#L90

@nickvergessen
Copy link
Member Author

@nextcloud/javascript should be fine to just replace it everywhere?

@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #2770 (Files External tests failing), #10330 (Files App: Layout issue when selecting multiple files), #3468 (Update detection issues for files external), #2962 (No icon for files and folders in file app), and #5142 (Files app not showing files).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants