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

Ability to add codecs via plugin system #1773

Closed
3 of 6 tasks
drewwestphal opened this issue Sep 29, 2023 · 3 comments · Fixed by #1949
Closed
3 of 6 tasks

Ability to add codecs via plugin system #1773

drewwestphal opened this issue Sep 29, 2023 · 3 comments · Fixed by #1949

Comments

@drewwestphal
Copy link

Feature description

It is currently not possible to add a custom scalar codec via the plugin system

Motivating example

Working on a project that has columns of type ltree and mutation function parameters of type ltree

#1772
https://discord.com/channels/489127045289476126/1157342633644003429

Supporting development

I [tick all that apply]:

  • am interested in building this feature myself
  • am interested in collaborating on building this feature
  • am willing to help testing this feature before it's released
  • am willing to write a test-driven test suite for this feature (before it exists)
  • am a Graphile sponsor ❤️
  • have an active support or consultancy contract with Graphile
@benjie
Copy link
Member

benjie commented Sep 29, 2023

Related: graphile/postgis#58

@benjie benjie added this to V5.0.0 Oct 4, 2023
@github-project-automation github-project-automation bot moved this to 🌳 Triage in V5.0.0 Oct 4, 2023
@benjie benjie moved this from 🌳 Triage to 🦟 Mayfly in V5.0.0 Oct 4, 2023
@jsmnbom
Copy link

jsmnbom commented Jan 29, 2024

I'm currently using a hacky Plugin like this (and excluding the native "PgCodecsPlugin". But having this built in would definitely be very helpful ^^
https://gist.github.com/jsmnbom/67c3be9c30b1df96ce652b9c23741c0e

[x] am interested in building this feature myself

tho I don't exactly know how it should look. Naively it is just implementing a hook inside getCodecFromType but, the t helper from the original source is quite useful and should maybe be exported in a more user friendly manner?

@benjie
Copy link
Member

benjie commented Jan 29, 2024

I'm getting back to V5 work again after a few months on other things (catching up, Worker, holidays, client work) and this is in my high priority list so leave it with me - I'll reach out if I can give some easy guidance.

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

Successfully merging a pull request may close this issue.

3 participants