Skip to content

Add ip_address to authenticated and failed authentication payloads#13

Merged
DanielRivers merged 1 commit intomainfrom
feat/add_ip_address_user_payload
Mar 17, 2026
Merged

Add ip_address to authenticated and failed authentication payloads#13
DanielRivers merged 1 commit intomainfrom
feat/add_ip_address_user_payload

Conversation

@DanielRivers
Copy link
Member

Explain your changes

Adds ip_address to payloads

Checklist

🛟 If you need help, consider asking for advice over in the Kinde community.

@DanielRivers DanielRivers requested a review from a team as a code owner March 16, 2026 17:31
@coderabbitai
Copy link

coderabbitai bot commented Mar 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 15b20158-7af5-47e7-b00f-11b4b0d1042e

📥 Commits

Reviewing files that changed from the base of the PR and between a6b5fe8 and 353da29.

📒 Files selected for processing (1)
  • lib/types.ts

Walkthrough

Adds ip_address: string field to two webhook event types (UserAuthenticatedWebhookEvent and UserAuthenticationFailedWebhookEvent) in the type definitions file.

Changes

Cohort / File(s) Summary
Webhook Event Type Extensions
lib/types.ts
Added ip_address: string field to UserAuthenticatedWebhookEvent and UserAuthenticationFailedWebhookEvent type definitions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and specifically describes the main change: adding ip_address fields to both authenticated and failed authentication webhook event payloads.
Description check ✅ Passed The description directly relates to the changeset by stating 'Adds ip_address to payloads' which aligns with the webhook event type modifications in lib/types.ts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/add_ip_address_user_payload
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can use Trivy to scan for security misconfigurations and secrets in Infrastructure as Code files.

Add a .trivyignore file to your project to customize which findings Trivy reports.

@codecov
Copy link

codecov bot commented Mar 16, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@DanielRivers DanielRivers merged commit d601c16 into main Mar 17, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants