Results 163 comments of Marc Busqué

@kennyadsl, in https://github.com/solidusio/solidus/commit/1f7442f3c44c16225c0ed24f5c515e2a189a77f5 we fixed a different issue. If the class is not present anymore, that's another problem. We should check whether psych supports something OOTB and, otherwise, rescue from...

Thanks for reporting, @diegomichel. Yeah, that's a known problem, but it's nice you created a dedicated issue as it'll have more visibility. See https://github.com/solidusio/solidus/pull/4231: > On a related information, the...

Copying the legacy guides over the corresponding classes will be an improvement of the current situation, but if we do that blindly, we face two problems: - The legacy guides...

@kennyadsl, thanks for getting back. Ideally, I'd like to see the yard docs as the "Reference" part in our documentation ecosystem, while the guides would cover "Tutorial", "How-to," & "Explanation"....

> Until v4 we should have some sort of configuration for switching between using the API endpoints and the the local backend controllers (unless https://github.com/orgs/solidusio/teams/core-team thinks differently) We can use...

I also suspected of that commit, but looking to the diff I don't see how it could break that 🤷 Thanks for looking into it, I'm curious about what's the...

🤦 Good caught! Feel free to submit a PR fixing it. I think 1 would be ok for now.

Probably the issue can be moved to the solidus_auth_devise repository, but I'll leave it here until further investigation.

To be honest, I don't know why. Happy to only keep the MIT one.