Skip to content

feat(module:message,notification): display nzData when content is a template#9001

Merged
Laffery merged 1 commit into
NG-ZORRO:masterfrom
Nicoss54:feature/nz-data-notification
Feb 19, 2025
Merged

feat(module:message,notification): display nzData when content is a template#9001
Laffery merged 1 commit into
NG-ZORRO:masterfrom
Nicoss54:feature/nz-data-notification

Conversation

@Nicoss54
Copy link
Copy Markdown
Collaborator

@Nicoss54 Nicoss54 commented Feb 8, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Application (the showcase website) / infrastructure changes
  • Other... Please describe:

What is the current behavior?

nzData are not passed when content is custom template

Issue Number: #8997

What is the new behavior?

nzData are passed when content is custom template

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@zorro-bot
Copy link
Copy Markdown

zorro-bot Bot commented Feb 8, 2025

This preview will be available after the AzureCI is passed.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.94%. Comparing base (d70a768) to head (26b0d31).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9001      +/-   ##
==========================================
+ Coverage   91.93%   91.94%   +0.01%     
==========================================
  Files         555      555              
  Lines       19708    19708              
  Branches     2946     2945       -1     
==========================================
+ Hits        18118    18120       +2     
+ Misses       1263     1262       -1     
+ Partials      327      326       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment thread components/message/typings.ts Outdated
Comment thread components/message/typings.ts Outdated
Comment thread components/notification/doc/index.en-US.md Outdated
Comment thread components/notification/doc/index.en-US.md Outdated
Comment thread components/notification/doc/index.en-US.md Outdated
Comment thread components/notification/doc/index.en-US.md Outdated
Comment thread components/notification/doc/index.zh-CN.md Outdated
Comment thread components/notification/typings.ts Outdated
Comment thread components/notification/notification.service.ts Outdated
Comment thread components/notification/notification.spec.ts Outdated
Comment thread components/notification/typings.ts Outdated
Comment thread components/notification/notification.service.ts Outdated
Comment thread components/notification/doc/index.en-US.md Outdated
Comment thread components/message/typings.ts Outdated
@Nicoss54
Copy link
Copy Markdown
Collaborator Author

@HyperLife1119 all seems done :)

@HyperLife1119 HyperLife1119 changed the title feat(module:notification): display nzData when content is a template feat(module:message,notification): display nzData when content is a template Feb 13, 2025
Comment thread components/message/demo/template.md Outdated
Comment thread components/message/demo/template.md Outdated
Comment thread components/message/doc/index.zh-CN.md Outdated
Comment thread components/notification/demo/custom-icon.ts Outdated
Comment thread components/notification/demo/template.ts Outdated
Comment thread components/notification/notification.service.ts Outdated
Comment thread components/notification/typings.ts Outdated
Copy link
Copy Markdown
Collaborator

@HyperLife1119 HyperLife1119 left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread components/notification/notification.service.ts Outdated
@Nicoss54 Nicoss54 force-pushed the feature/nz-data-notification branch from e06201a to 26b0d31 Compare February 19, 2025 18:25
@Nicoss54
Copy link
Copy Markdown
Collaborator Author

@Laffery your comments is taken into consideration and it's implemented

Copy link
Copy Markdown
Collaborator

@Laffery Laffery left a comment

Choose a reason for hiding this comment

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

LGTM

@Laffery Laffery merged commit 5157470 into NG-ZORRO:master Feb 19, 2025
@Laffery
Copy link
Copy Markdown
Collaborator

Laffery commented Feb 19, 2025

close #8997

@Nicoss54 Nicoss54 deleted the feature/nz-data-notification branch March 5, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants