Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Quick fix for new files OperationProgressbar #502

Merged
merged 2 commits into from
Mar 5, 2019

Conversation

juliusknorr
Copy link
Member

With the new OperationProgressbar the gallery app was not loading properly as the required scripts from the files app were missing.

I'm not to happy with all the files app dependencies in here, but this at least brings the gallery app back to being functional.

@juliusknorr juliusknorr added this to the Nextcloud 16 milestone Feb 26, 2019
@juliusknorr juliusknorr added bug Something isn't working 3. to review Waiting for reviews high High priority labels Feb 26, 2019
@MorrisJobke MorrisJobke mentioned this pull request Mar 4, 2019
45 tasks
@juliusknorr
Copy link
Member Author

@MorrisJobke @rullzer Would be good to have this in beta1, otherwise the gallery app is broken.

@juliusknorr
Copy link
Member Author

Hm seems the merged javascript has a weired race condition, it works in debug mode and when loading the merged file from the browser cache, but not on hard reload.

@juliusknorr juliusknorr added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Mar 5, 2019
@juliusknorr juliusknorr force-pushed the bugfix/noid/fix-upload branch from 119ceef to 976f13d Compare March 5, 2019 17:47
Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@MorrisJobke MorrisJobke merged commit 80e9458 into master Mar 5, 2019
@MorrisJobke MorrisJobke deleted the bugfix/noid/fix-upload branch March 5, 2019 17:57
@MorrisJobke MorrisJobke mentioned this pull request Mar 6, 2019
9 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2. developing Work in progress bug Something isn't working high High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants