feat: support upserting tables with only unique constraint(s)

[bchrobot]

Problem

Upsert mutations are not created for tables without primary keys, even if they have at least one unique constraint. This is inconsistent with Postgraphile core, which generates update mutations for tables with just unique constraints.

Proposed Solution

This updates logic to check for primary key OR at least one unique constraint on the table.

Testing

I have extended an existing test case but have not run the test suite yet (a Contributing section in readme would be a nice addition 🙂).

I have verified behavior experimentally by yarn linking the local clone to an application codebase.

[cdaringe]

Githib should show ya the contributing.md, but a link would be nice too. Feel free to add those patches inline here.

https://github.com/cdaringe/postgraphile-upsert/blob/main/.github/contributing.md

missing is the call fir need of docker for integration tests to run

[bchrobot]

Githib should show ya the contributing.md, but a link would be nice too. Feel free to add those patches inline here.

https://github.com/cdaringe/postgraphile-upsert/blob/main/.github/contributing.md

missing is the call fir need of docker for integration tests to run

Ah! Thank you for the link. I hadn't thought to look in the .github folder. I don't recall being prompted when opening the PR but maybe missed it

[bchrobot]

Okay, I got tests running and passing locally now

[cdaringe]

Thanks!

[cdaringe]

Ah crap i meant to release it as a breaking change

[github-actions]

🎉 This PR is included in version 3.3.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[bchrobot]

Ah crap i meant to release it as a breaking change

There isn't really a way to pass config options to v4 plugins is there? To have this feature there but off by default.

[cdaringe]

No big deal :) thx for the patches!