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

User on composer based sites getting errors #546

Closed
W3ap0n-X opened this issue Feb 10, 2023 · 5 comments · Fixed by #583
Closed

User on composer based sites getting errors #546

W3ap0n-X opened this issue Feb 10, 2023 · 5 comments · Fixed by #583

Comments

@W3ap0n-X
Copy link
Contributor

More details in BG Premium ticket #21418

@timnolte
Copy link

This plugin doesn't properly use WordPress Core methods to generate assets paths, specifically like here:

https://github.com/BoldGrid/boldgrid-backup/blob/master/admin/class-boldgrid-backup-admin-settings.php#L1026

This causes major loading issues for all CSS & JS assets when WordPress Core is installed in an alternative path.

@timnolte
Copy link

Seeing that this looks to be an issue with the https://github.com/BoldGrid/library package not properly using WordPress Core methods for paths.

@jamesros161
Copy link
Contributor

@timnolte Thank you for the changes suggested. I have tested the changes you've requested, and it seems to be working as expected. However, I wanted to see if you'd like to test it before I deploy it to stable. The Total Upkeep 1.15.10 can be downloaded here

@timnolte
Copy link

@jamesros161 thanks, I'll try testing this out as soon as I can. Thanks!

@timnolte
Copy link

@jamesros161 Hmm, so I'm not certain if this is an issue with me site setup somehow or and issue between this version of the plugin and the premium version of the plugin but I can't seem to authenticate with Google Drive with the 1.15.10 version of the plugin even thought generally things look to be working. I'm getting a 403 error in regards to a bad nonce.

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

Successfully merging a pull request may close this issue.

3 participants