-
Notifications
You must be signed in to change notification settings - Fork 258
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: support ci workflow notifications #766
Conversation
Notification title parsing was required since the Notification Event payload doesn't contain any status, branch or workflow name details. Similarly, I couldn't find an API to query for these given a notification id |
#793 to be merged prior to this enhancement |
Found an API that looks promising. Will explore that route rather than string regexing |
Although the Check Suite API would be a perfect fit our needs, there is no joining identifier (that i can see) within the List Notifications response to be able to make it useful. What we would have to do call List Workflow Runs for a Repository and parse the results, in order to get conclusion value etc. With this in mind, I suggest for now we continue down the path of inferring the key values based on the workflow title |
Closes #381 and #526