fix: do validation for params off constructor so it can throw before starting

electric-sql / electric

Sync little subsets of your Postgres data into local apps and services.

https://electric-sql.com

Apache License 2.0

6.47k stars 156 forks source link

Closed KyleAMathews closed 3 days ago

KyleAMathews commented 6 days ago

did this wrong in #1978 so that combined with #1983, passing a reserved parameter just means it silently fails atm.

netlify[bot] commented 3 days ago

Name	Link
Latest commit	22c1dcfa5b5cf2fc9e753631ec0f67b99ddde770
Latest deploy log	https://app.netlify.com/sites/electric-next/deploys/673bac00d4180200082883ad
Deploy Preview	https://deploy-preview-1984--electric-next.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.