-
Notifications
You must be signed in to change notification settings - Fork 33
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
feat: add support for Cloudflare Turnstile response param in signUp (#155) #156
Conversation
561d4d4
to
a367317
Compare
Thanks for the PR, just a minor comment, otherwise, LGTM :) |
@dbarrosop I implemented your recommendations, let me know if you wnat anything else please |
Could you allow edits and let me know so I can fix the CI? |
@dbarrosop 🤔 It was already enabled... |
I will never understand github...
anyway, it is a simple fix so maybe it is faster if you do it yourself, edit the file diff --git a/.github/workflows/test.nhost_auth_dart.yaml b/.github/workflows/test.nhost_auth_dart.yaml
index afacf73..8ad5a5f 100644
--- a/.github/workflows/test.nhost_auth_dart.yaml
+++ b/.github/workflows/test.nhost_auth_dart.yaml
@@ -22,7 +22,8 @@ on:
# Run the quality job at 4am every day
# - cron: '0 4 * * *'
-env: {}
+env:
+ MAX_PANA_MISSING_POINTS: 40
jobs:
# Ensure that the package meets a suitably high pub.dev score
score-package-quality: |
@dbarrosop patch applied 🙂 |
Thanks! Merging. |
Fixes #155