-
Notifications
You must be signed in to change notification settings - Fork 215
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 HTTP hook Secret Configuration #2129
Conversation
27d20e3
to
3b366f9
Compare
Pull Request Test Coverage Report for Build 8998328342Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
Actually I think we’ll need to wait for the FE/BE/docs to go out before merging :/ |
b027946
to
cb071e2
Compare
3dfd9cc
to
1a40588
Compare
fd978a1
to
476cf01
Compare
New Changes
The backend is now merged and I'm hoping to get this in so that we can allow contributors to write examples for custom SMS providers How this was tested?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might want to add a valid config to init_config.test.toml
as a sanity check.
76698f6
to
63f0f7c
Compare
What kind of change does this PR introduce?
Adds the corresponding configuration for the
SendSMS
andSendEmail
HTTP Hooks. Also introduces Secrets for previous hooks -custom_access_token
,mfa_verification_attempt
,password_verification_attempt
Backend PR (Merged): https://github.com/supabase/infrastructure/pull/17848
Frontend PR (WIP): supabase/supabase#22495