Skip to content

feat(backend): also publish webhooks to operators if primary recipient is tenant #3368

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 22 commits into
base: 2893/multi-tenancy-v1
Choose a base branch
from
Open

feat(backend): also publish webhooks to operators if primary recipient is tenant #3368

wants to merge 22 commits into from

Conversation

njlie
Copy link
Contributor

@njlie njlie commented Apr 2, 2025 •
edited
Loading

Changes proposed in this pull request

  • Publishes a second webhook to the operator's webhook URL if primary recipient is a tenant of the Rafiki instance.

Context

Fixes #3283.

Checklist

  • Related issues linked using fixes #number
  • Tests added/updated
  • Make sure that all checks pass
  • Bruno collection updated (if necessary)
  • Documentation issue created with user-docs label (if necessary)
  • OpenAPI specs updated (if necessary)

@github-actions github-actions bot added type: tests Testing related pkg: backend Changes in the backend package. type: source Changes business logic labels Apr 2, 2025
@njlie njlie force-pushed the nl/3121/tenanted-webhooks branch from 7cfa385 to 14fe53f Compare April 2, 2025 22:28
@njlie njlie force-pushed the nl/3283/operator-webhooks branch 2 times, most recently from a5ce199 to 98b1291 Compare April 8, 2025 17:59
@njlie njlie force-pushed the nl/3121/tenanted-webhooks branch from 4260036 to 5c51d6d Compare April 8, 2025 18:45
@njlie njlie force-pushed the nl/3283/operator-webhooks branch from 98b1291 to 64973e7 Compare April 8, 2025 20:20
Base automatically changed from nl/3121/tenanted-webhooks to 2893/multi-tenancy-v1 April 15, 2025 23:13
@github-actions github-actions bot added pkg: frontend Changes in the frontend package. pkg: mock-account-service-lib labels Apr 17, 2025
golobitch and others added 19 commits April 17, 2025 14:49
@golobitch @njlie
tests(tenants): when tenant is created, default settings should be set
d3ffb83
@golobitch @njlie
feat(graphql): tenant settings
ccf1a0c
@golobitch @njlie
fix(tenantSettings): address PR comments
85cb7a4
@golobitch @njlie
fix(tenantSettings): address PR comments
d4bfa8c
@njlie
feat(backend): tenanted webhooks
73498b6
@njlie
fix: rebase
85b1bce
@njlie
chore: fix some rebasing issues
13ad1d0
@njlie
feat: add tenant id to tests; temporary operator tenant id in peer we…
a5b280d 
…bhook events
@njlie
fix: build errors
8a50f42
@njlie
feat: remove temporary tenantId code from before tenanted peers
ce6bef7
@njlie
fix: build errors
8902c3e
@golobitch @njlie
tests(tenants): when tenant is created, default settings should be set
2ded8d2
@golobitch @njlie
fix(tenantSettings): address PR comments
f9b053a
@njlie
feat(backend): tenanted webhooks
309e42b
@njlie
chore: fix some rebasing issues
7a8426d
@njlie
feat(backend): also publish webhooks to operators if primary recipien…
148de4a 
…t is tenant
@njlie
fix: rebase issues
b844660
@njlie
fix: tests
bd66536
@njlie
feat: add webhook model
659f2ee
@njlie njlie force-pushed the nl/3283/operator-webhooks branch from 1950deb to d241f0f Compare April 17, 2025 21:54
@njlie njlie marked this pull request as ready for review April 18, 2025 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BlairCurrey BlairCurrey Awaiting requested review from BlairCurrey

@mkurapov mkurapov Awaiting requested review from mkurapov

@golobitch golobitch Awaiting requested review from golobitch

@sanducb sanducb Awaiting requested review from sanducb

@oana-lolea oana-lolea Awaiting requested review from oana-lolea

Assignees
No one assigned
Labels
pkg: backend Changes in the backend package. pkg: frontend Changes in the frontend package. pkg: mock-account-service-lib pkg: mock-ase type: source Changes business logic type: tests Testing related
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@njlie @golobitch