Skip to content

We need to ensure that the app holding the objectstore implementation… - #2113

Merged
rullzer merged 1 commit into
masterfrom
downstream-26299
Nov 14, 2016
Merged

We need to ensure that the app holding the objectstore implementation…#2113
rullzer merged 1 commit into
masterfrom
downstream-26299

Conversation

@MorrisJobke

Copy link
Copy Markdown
Member

… is loaded - fixes owncloud/core#26299

  • also fixes the setup issue with other objectstores
  • steps: configure objectstore in config.php before installing, then run the installer

cc @icewind1991 @rullzer

@icewind1991 or did you fixed this differently in your objectstore implementation? This helps to load the apps that are not shipped in core by default.

@MorrisJobke MorrisJobke added this to the Nextcloud 11.0 milestone Nov 14, 2016
@mention-bot

Copy link
Copy Markdown

@MorrisJobke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @LukasReschke, @PVince81 and @DeepDiver1975 to be potential reviewers.

@rullzer

rullzer commented Nov 14, 2016

Copy link
Copy Markdown
Member

Uhm well... we don't have external ObjectStore apps. But yeah sure. LGTM

@MorrisJobke

Copy link
Copy Markdown
Member Author

Uhm well... we don't have external ObjectStore apps. But yeah sure. LGTM

Yes, but we encourage people to create one and without this it is not possible to install or upgrade an instance that is using such an app.

👍 from me as well

@MorrisJobke MorrisJobke added 3. to review Waiting for reviews downstream 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 14, 2016
@rullzer
rullzer merged commit 1e3d806 into master Nov 14, 2016
@rullzer
rullzer deleted the downstream-26299 branch November 14, 2016 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants