Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Notifier][Smsbox] Add webhook support#59570

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
OskarStark merged 1 commit intosymfony:7.3fromsmsbox-net:smsbox-notifier
Feb 28, 2025

Conversation

alanzarli
Copy link

@alanzarlialanzarli commentedJan 21, 2025•
edited
Loading

QA
Branch?7.3
Bug fix?no
New feature?yes
Deprecations?no
LicenseMIT
Doc#20590

Add support for Webhook to Smsbox Notifier
Event statuses come from Smsboxdocumentation.

@alanzarlialanzarliforce-pushed thesmsbox-notifier branch 3 times, most recently fromc0ecd8b to95bb117CompareJanuary 31, 2025 15:58
Copy link
Member

@fabpotfabpot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

You need to update the deps for framework bundle to fix the broken tests.

alanzarli reacted with thumbs up emoji
@alanzarli
Copy link
Author

done 👍

@OskarStark
Copy link
Contributor

OskarStark commentedFeb 28, 2025•
edited
Loading

Thanks Alan for working on this feature, this is much appreciated and congrats on your first contribution to the Symfony framework 🎉 👏

alanzarli reacted with heart emoji

@OskarStarkOskarStark merged commit35202a2 intosymfony:7.3Feb 28, 2025
9 of 11 checks passed
@OskarStarkOskarStark changed the title[Notifier][Webhook] Add Smsbox support[Notifier][Smsbox] Add webhook supportFeb 28, 2025
@smsbox-netsmsbox-net deleted the smsbox-notifier branchFebruary 28, 2025 13:29
@fabpotfabpot mentioned this pull requestMay 2, 2025
OskarStark added a commit that referenced this pull requestMay 16, 2025
This PR was merged into the 7.3 branch.Discussion----------[Notifier] simplify Webhook constraint| Q             | A| ------------- | ---| Branch?       | 7.3| Bug fix?      | no| New feature?  | no| Deprecations? | no| Issues        || License       | MITintroduced in#59570Commits-------193b69c simplify Webhook constraint
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@welcoMatticwelcoMatticwelcoMattic approved these changes

@OskarStarkOskarStarkOskarStark approved these changes

@chalasrchalasrAwaiting requested review from chalasr

@fabpotfabpotAwaiting requested review from fabpot

Assignees
No one assigned
Projects
None yet
Milestone
7.3
Development

Successfully merging this pull request may close these issues.

5 participants
@alanzarli@OskarStark@fabpot@welcoMattic@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp