Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upEnable custom texts for notifications #3531
Comments
|
Hi! I've implemented a rough draft of what this could look like in #3534 but I think some discussion on the correct API for this would be prudent! Please let me know if something like that would fit your use-case. @joshmeek thoughts? This will take additional work to add to all of the notifications and update documentation, this would be a good issue for someone from the community! |
|
@madkinsz I'm guessing we would want the template to be "templatable" so I think something similar to what we do for task targets would be a nice implementation. Task targets can take in templatable strings like |
|
Hi all, there are some comments in the relevant PoC PR and this is ready for anyone in the community to work on. |
|
Hey @madkinsz, that looks already like a great framework in which to bring this functionality to live indeed! Unfortunately, I am still pretty busy at the moment, but if nobody else looks into this, I might be able to finish it in ~10 days :) |
Current behavior
Slack, email and other notifications provide a standardized text and backend info.
Proposed behavior
Users can add custom texts to the notifications.
Example
Ideas for Implementation
add kwargs to functions in notifications.py to support a "custom_text"