graphqlite-bundle icon indicating copy to clipboard operation
graphqlite-bundle copied to clipboard

[Error handling] Remove category from exceptions

Open cvergne opened this issue 1 year ago • 2 comments

Apply changes from https://github.com/thecodingmachine/graphqlite/pull/685 to the bundle.

[!WARNING] Do no not merge until merge and release of https://github.com/thecodingmachine/graphqlite/pull/685

[!NOTE] Still having error running test about DependencyInjection/Configuration.php not compatible with implemented interface. Not fixed it in this PR (as already done in #208), but should probably split it into a distinct PR.

cvergne avatar May 09 '24 09:05 cvergne

Hello @cvergne thank you for this contribution! Can you update the code to fix the pipeline?

homersimpsons avatar Jun 08 '24 15:06 homersimpsons

Hello @homersimpsons The PR requires the one from the main library to be released to work, which will fix the pipeline ^^'

That's why I left the PR in draft as it's not ready.

cvergne avatar Jun 10 '24 14:06 cvergne

The upstream pull request has been merged. This one should most likely be merged (but needs an upgrade of graphqlite library)

homersimpsons avatar Dec 14 '24 19:12 homersimpsons

need an update of library

@homersimpsons this is not so easy https://github.com/thecodingmachine/graphqlite-bundle/pull/203#issuecomment-2484381126

andrew-demb avatar Dec 14 '24 19:12 andrew-demb

Replaced by https://github.com/thecodingmachine/graphqlite-bundle/pull/229

Thank you @cvergne

andrew-demb avatar Dec 31 '24 15:12 andrew-demb