Skip to content
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

Implement Webhooks #548

Merged
merged 49 commits into from
Oct 27, 2024
Merged

Implement Webhooks #548

merged 49 commits into from
Oct 27, 2024

Conversation

colindecarlo
Copy link
Contributor

@colindecarlo colindecarlo commented Aug 14, 2024 •
edited by lancepioch
Loading

Changes:

  • Remove unused Server, Subuser, User, EggVariable observers
  • Remove unused Server, Subuser, and User events
  • Add WebhookConfiguration model to store where webhooks should be sent
  • Add Webhook model to store previously executed webhooks
  • Add Webhook Resource for Filament
  • Add event new event listener (DispatchWebhooks) for eloquent model changes and our own app's events
  • Fix weird issue with unused field_type for eggs

PascaleBeier and others added 20 commits August 9, 2024 16:13
@PascaleBeier @go-vehikl
feat: First Webhook PoC draft
fc1c51e
@PascaleBeier @go-vehikl
feat: Dispatch Webhooks PoC
ca1004d
@PascaleBeier @go-vehikl
fix: typo in webhook configuration scope
b710dd7
@PascaleBeier @lancepioch @go-vehikl
Update 2024_04_21_162552_create_webhooks_table.php
b458612 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update 2024_04_21_162552_create_webhooks_table.php
72a2c3c 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update 2024_04_21_162544_create_webhook_configurations_table.php
cd4aaf6 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update 2024_04_21_162544_create_webhook_configurations_table.php
b66ca7a 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhooks.php
4e1ecac 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhooksJob.php
db7ffd4 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhookForConfiguration.php
73507f2 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhookForConfiguration.php
e5d08d9 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhookForConfiguration.php
538d46f 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhooksJob.php
dcfe61a 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhooksJob.php
afeb81a 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @lancepioch @go-vehikl
Update DispatchWebhooksJob.php
c7667e3 
Co-authored-by: Lance Pioch <[email protected]>
@PascaleBeier @go-vehikl
chore: Implement Webhook Event Discovery
f0e1f75
@go-vehikl
we got a test working for webhooks
ebe40d1
@go-vehikl
WIP
74b6085
@go-vehikl
Something is working!
63c917d
@colindecarlo
Merge branch 'pelican-dev:main' into feat-webhooks
ad638d9
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 14, 2024 •
edited
Loading

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@pelican-dev pelican-dev deleted a comment from Poseidon281 Aug 16, 2024
@notAreYouScared
Copy link
Member

If you wish to continue this, please open a new PR.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 7, 2024
@lancepioch lancepioch self-assigned this Oct 20, 2024
@pelican-dev pelican-dev unlocked this conversation Oct 24, 2024
@lancepioch lancepioch marked this pull request as ready for review October 24, 2024 20:52
@lancepioch lancepioch requested review from Boy132, lancepioch and notAreYouScared and removed request for Boy132 October 24, 2024 20:52
@RMartinOscar RMartinOscar linked an issue Oct 26, 2024 that may be closed by this pull request
Implement Webhooks #9
Closed
@notAreYouScared notAreYouScared dismissed lancepioch’s stale review October 27, 2024 00:34

Lance said so. ~lance

@notAreYouScared notAreYouScared merged commit 86c369d into pelican-dev:main Oct 27, 2024
15 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 27, 2024
@pelican-dev pelican-dev deleted a comment from colindecarlo Oct 31, 2024
@pelican-dev pelican-dev deleted a comment from go-vehikl Oct 31, 2024
@RMartinOscar RMartinOscar modified the milestone: beta11 Nov 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@notAreYouScared notAreYouScared notAreYouScared approved these changes

@lancepioch lancepioch lancepioch left review comments

@Boy132 Boy132 Awaiting requested review from Boy132

Assignees

@lancepioch lancepioch

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement Webhooks
6 participants
@colindecarlo @notAreYouScared @lancepioch @RMartinOscar @PascaleBeier @go-vehikl