-
Notifications
You must be signed in to change notification settings - Fork 19
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
TOS breaks PicoCMS #274
Comments
@nickvergessen @PhrozenByte I will lift this issue as important to solve. I want both social login and PicoCMS |
We use it to be able to use markdown in your terms of service, so yes kind of. |
@ChristophWurst Subscribed to your issue for updates. Since it's an ecosystem issue, are we going to see this being officially supported by Nextcloud? IMO it would be a good thing to give @coenjacobs some support (and some visibility) by Nextcloud, as it looks like he's doing this on his own... It's neither just Guzzle nor Parsedown, it's kind of a miracle that these kind of conflicts don't happen more often... |
Well every app can/should do this when it has such problems. It's nothing Nextcloud can fix for all app maintainers |
@nickvergessen It's all about recommendations and documentation 😉 |
I want to explore these tools a bit more. If they can be used reliably, we can add them to our docs. While it currently doesn't help with Guzzle conflicts, it could work with parsedown. You may look at my PR and just try to apply it here. Then check if the app still works :) |
"Call to undefined method ParsedownExtra::textElements()" in PicoCMS with TOS enabled.
Please reference the thread on PicoCMS about this issue.
nextcloud/cms_pico#97
The text was updated successfully, but these errors were encountered: