search

alsciende / fiveringsdb-api

Deckbuilder for Legend of the Five Rings (API server)
https://fiveringsdb.com
MIT License
2 stars 4 forks source link

Added serializer back in #89

Closed xWorkerBeex closed 5 years ago

alsciende commented 5 years ago

This is useless. The package is on packagist: https://packagist.org/packages/alsciende/serializer-bundle.

fatihi commented 5 years ago

Ah good to know, thank you! There is an issue with saving the decks and it came from your commit, so we were basically just trying to figure out what caused it

fatihi commented 5 years ago

Does that mean we can just add the package to "require"?

alsciende commented 5 years ago

Yes, like every other packages, it just needs to be required and composed will find it automatically through packagist. That’s how I updated composer.lock, and it works.

I don’t know why there’s an issue with deck saving though...

alsciende commented 5 years ago

I didn’t run the behat tests. That could give us a hint.