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

Fixes #111: Multisite issue with main site #113

Merged
merged 1 commit into from Jan 31, 2018
Merged

Fixes #111: Multisite issue with main site #113

merged 1 commit into from Jan 31, 2018

Conversation

JonasBrand
Copy link
Contributor

@JonasBrand JonasBrand commented Jan 31, 2018

When using WP Document Revisions in a multisite installation, there were issues with the uploads folder setting when using the plugin both on the main site (usually site with ID = 1) and on other sub-sites. This pull request fixes that by removing the site-specific part for the main site.

@request-info
Copy link

request-info bot commented Jan 31, 2018

Thanks for this. Do you mind providing a bit more information about what problem you're trying to solve?

@JonasBrand
Copy link
Contributor Author

When using WP Document Revisions in a multisite installation, there were issues with the uploads folder setting when using the plugin both on the main site (usually site with ID = 1) and on other sub-sites. This pull request fixes that by removing the site-specific part for the main site.

@benbalter benbalter merged commit f9b43b1 into wp-document-revisions:master Jan 31, 2018
@welcome
Copy link

welcome bot commented Jan 31, 2018

Congrats on getting your first pull request to WP Document Revisions merged! Without amazing humans like you submitting pull requests, we couldn’t run this project. You rock! 🎉

If you're interested in tackling another bug or feature, take a look at the open issues, especially those labeled help wanted.

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

Successfully merging this pull request may close these issues.

None yet

2 participants