Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Add notification get command #2818

Merged
merged 3 commits into from
Feb 9, 2023

Conversation

tevoinea
Copy link
Member

@tevoinea tevoinea commented Feb 8, 2023

Summary of the Pull Request

What is this about?

Closes #2817. Adds support for onefuzz notification get <notification_id>

Info on Pull Request

What does this include?

  • Adds the new command

Validation Steps Performed

How does someone test & validate?

Ran the command on a dev instance to get a notification.

@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2023

Codecov Report

Merging #2818 (73bde28) into main (a13bb2f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2818   +/-   ##
=======================================
  Coverage   29.93%   29.93%           
=======================================
  Files         324      324           
  Lines       38992    38992           
=======================================
  Hits        11671    11671           
  Misses      27321    27321           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tevoinea tevoinea merged commit 2e6eef7 into microsoft:main Feb 9, 2023
@mgreisen mgreisen mentioned this pull request Mar 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for onefuzz notifications get <notification_id>
3 participants