Pieter Cardillo Kwok
Pieter Cardillo Kwok
> Could the topic and template just show that string "N/A"? Yeah, that looks nicer than just "-" 
@laurynmm, Thank you for the detailed review and feedback. I’ve pushed some more changes to address your comments. Most of the changes are self-explanatory, but for those that needed more...
>In the fallback message, when expected fields are missing, I wonder if we should include the keys that are missing in that message text. Since it's likely these updates will...
>Why are there fields in the documentation example payload that we're ignoring when creating the Zulip message? For example impactedEntities, trigger, etc.. I feel like we should either put these...
@laurynmm The revision looks great. I’ve pushed some additional changes to address your comments and updated the screenshots in the PR description. Let me know if there’s anything else we...
Update: Cleaned up some test fixture files to address @laurynmm 's comment. I've also updated the doc screenshot in the PR description
Update: New discussion in this [CZO](https://chat.zulip.org/#narrow/stream/101-design/topic/.2329730.20Updating.20Newrelic.20Notification) for the new notification message design
Update: #29730 is ready for review!
@zulipbot add "buddy review"