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][Novu] Implement overrides#51577

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

Conversation

@wouter-toppy
Copy link
Contributor

@wouter-toppywouter-toppy commentedSep 6, 2023•
edited by nicolas-grekas
Loading

QA
Branch?6.4
Bug fix?no
New feature?yes
Deprecations?no
TicketsFix#51576
LicenseMIT

ImplementNovu Overrides

@carsonbotcarsonbot added this to the6.4 milestoneSep 6, 2023
@wouter-toppywouter-toppy changed the titleImplement Novu overrides[RFC] Implement Novu overridesSep 6, 2023
@wouter-toppywouter-toppyforce-pushed thefeat-novu-overrides branch 2 times, most recently froma3a5e80 toba31143CompareSeptember 6, 2023 10:15
@stloyd
Copy link
Contributor

The title should be changed, it's not RFC but[Notifier]

@carsonbotcarsonbot changed the title[RFC] Implement Novu overridesImplement Novu overridesSep 6, 2023
@carsonbotcarsonbot changed the titleImplement Novu overrides[Notifier] Implement Novu overridesSep 6, 2023
@OskarStark
Copy link
Contributor

Can you please add a test case including overrides and not just an empty array? Thanks

wouter-toppy reacted with thumbs up emoji

@OskarStarkOskarStark changed the title[Notifier] Implement Novu overrides[Notifier][Novu] Implement overridesOct 20, 2023
@nicolas-grekasnicolas-grekas modified the milestones:7.1,6.4Oct 20, 2023
@nicolas-grekas
Copy link
Member

Thank you@wouter-toppy.

wouter-toppy reacted with heart emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@stofstofAwaiting requested review from stof

@OskarStarkOskarStarkAwaiting requested review from OskarStarkOskarStark is a code owner

1 more reviewer

@stloydstloydstloyd approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

6.4

Development

Successfully merging this pull request may close these issues.

[Notifier] Implement Novu notifier overrides

8 participants

@wouter-toppy@stloyd@OskarStark@nicolas-grekas@stof@fabpot@derrabus@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp